git.delta.rocks / jrsonnet / refs/commits / 31f2b649c192

difftreelog

Enable multi-occurences for --jpath + improve positional params error message

Loris FRIEDEL2021-11-03parent: #afbb8dc.patch.diff
in: master

2 files changed

modifiedcrates/jrsonnet-cli/src/lib.rsdiffbeforeafterboth
--- a/crates/jrsonnet-cli/src/lib.rs
+++ b/crates/jrsonnet-cli/src/lib.rs
@@ -51,7 +51,7 @@
 	/// Any not found `imported` file will be searched in these.
 	/// This can also be specified via `JSONNET_PATH` variable,
 	/// which should contain a colon-separated (semicolon-separated on Windows) list of directories.
-	#[clap(long, short = 'J')]
+	#[clap(long, short = 'J', multiple_occurrences = true)]
 	jpath: Vec<PathBuf>,
 }
 impl ConfigureState for MiscOpts {
modifiedcrates/jrsonnet-parser/src/lib.rsdiffbeforeafterboth
62 let mut defaults_started = false;62 let mut defaults_started = false;
63 for param in &params {63 for param in &params {
64 defaults_started = defaults_started || param.1.is_some();64 defaults_started = defaults_started || param.1.is_some();
65 assert_eq!(defaults_started, param.1.is_some(), "defauld parameters should be used after all positionals");65 assert_eq!(defaults_started, param.1.is_some(), "defauld parameters should be used after all positionals. Misplaced default parameter: {:?}", param.0);
66 }66 }
67 expr::ParamsDesc(Rc::new(params))67 expr::ParamsDesc(Rc::new(params))
68 }68 }