Skip to content
This repository was archived by the owner on Aug 16, 2021. It is now read-only.

Allow using * in textual CLI parameters #82

Merged
merged 1 commit into from
Aug 17, 2018
Merged

Conversation

NikolayS
Copy link
Collaborator

@NikolayS NikolayS commented Aug 12, 2018

Now this works:

nancy run \
  --db-dump "create table t1 as select * from generate_series(1, 1000);" \
  --workload-custom-sql "select count(*) from t1;"

Additionally:

  • fix gitignore to ignore .gz and .json
  • ignore tmp dir in ./docker
  • code style in ./nancy

fix gitignore to ignore .gz and .json

ignore tmp dir in ./docker
@NikolayS NikolayS requested a review from dmius August 12, 2018 22:52
@NikolayS NikolayS assigned NikolayS and dmius and unassigned NikolayS Aug 12, 2018
@NikolayS NikolayS changed the title Allow using * in CLI options #48 Allow using * in textual CLI parameters Aug 12, 2018
@dmius
Copy link
Collaborator

dmius commented Aug 13, 2018

А что то я никаких изменений, кроме стилевых не вижу... @NikolayS куда смотреть?

@NikolayS
Copy link
Collaborator Author

@dmius Главный трюк: eval "$cmd" — кавычки здесь исправили проблему

@dmius dmius merged commit 75fe90b into master Aug 17, 2018
@dmius dmius deleted the fix_macos_and_wildcards branch August 17, 2018 13:52
NikolayS pushed a commit that referenced this pull request Sep 18, 2018
Allow using * in textual CLI parameters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants