Skip to content

Commit

Permalink
feat: disable empty list feature
Browse files Browse the repository at this point in the history
Signed-off-by: peefy <xpf6677@163.com>
  • Loading branch information
Peefy committed Nov 14, 2023
1 parent 55ad0ee commit aed9ce8
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 9 deletions.
8 changes: 4 additions & 4 deletions kclvm/api/src/service/service_impl.rs
Original file line number Diff line number Diff line change
Expand Up @@ -479,12 +479,12 @@ impl KclvmServiceImpl {
/// assert_eq!(result.changed_files.len(), 1);
/// ```
pub fn rename(&self, args: &RenameArgs) -> anyhow::Result<RenameResult> {
let symbol_path = args.symbol_path.clone();
let file_paths = args.file_paths.clone();
let new_name = args.new_name.clone();
let _symbol_path = args.symbol_path.clone();
let _file_paths = args.file_paths.clone();
let _new_name = args.new_name.clone();
Ok(RenameResult {
//todo: mock implementation
changed_files: file_paths,
changed_files: _file_paths,
})
}

Expand Down
1 change: 0 additions & 1 deletion kclvm/parser/src/entry.rs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ use kclvm_utils::path::{is_absolute, is_dir, path_exist};
use std::collections::VecDeque;
use std::fs;
use std::path::Path;
use std::sync::Arc;

use crate::LoadProgramOptions;

Expand Down
2 changes: 1 addition & 1 deletion kclvm/parser/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ use indexmap::IndexMap;
use kclvm_ast::ast;
use kclvm_config::modfile::{get_vendor_home, KCL_FILE_EXTENSION, KCL_FILE_SUFFIX, KCL_MOD_FILE};
use kclvm_error::diagnostic::Range;
use kclvm_error::{ErrorKind, Level, Message, Style};
use kclvm_error::{ErrorKind, Message, Style};
use kclvm_sema::plugin::PLUGIN_MODULE_PREFIX;
use kclvm_utils::path::PathPrefix;
use kclvm_utils::pkgpath::parse_external_pkg_name;
Expand Down
4 changes: 2 additions & 2 deletions kclvm/runtime/src/api/kclvm.rs
Original file line number Diff line number Diff line change
Expand Up @@ -327,8 +327,8 @@ pub struct ContextConfig {
pub list_option_mode: bool,
/// Whether to emit none value in the plan process.
pub disable_none: bool,
/// Whether to output empty list in the plan process.
pub plan_empty_list: bool,
/// Whether to emit empty list in the plan process.
pub disable_empty_list: bool,
}

#[derive(PartialEq, Eq, Clone, Debug)]
Expand Down
2 changes: 1 addition & 1 deletion kclvm/runtime/src/value/val_plan.rs
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ fn filter_results(ctx: &Context, key_values: &ValueRef, opts: &PlanOptions) -> V
let schema_in_list_count = ignore_schema_count + standalone_list.len();
let value = &value.as_list_ref().values;
// Plan empty list to values.
if value.is_empty() && ctx.cfg.plan_empty_list {
if value.is_empty() && !ctx.cfg.disable_empty_list {
let result = results.get_mut(0).unwrap();
result.dict_update_key_value(key.as_str(), ValueRef::list(None));
}
Expand Down
1 change: 1 addition & 0 deletions test/grammar/expr/select_expr/in_for_0/stdout.golden
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
c:
- name: main
volumeMounts: []

0 comments on commit aed9ce8

Please sign in to comment.