Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Provide a custom runnerfile using a flag #7

Closed
stylemistake opened this issue Sep 21, 2015 · 5 comments
Closed

Provide a custom runnerfile using a flag #7

stylemistake opened this issue Sep 21, 2015 · 5 comments
Labels
cli feature

Comments

@stylemistake
Copy link
Owner

Use file as a runnerfile:
runner -f <file>
@stylemistake stylemistake changed the title Provide custom runnerfile using a flag Provide a custom runnerfile using a flag May 17, 2016
stylemistake added a commit that referenced this issue May 24, 2016
@stylemistake
Implement flag parsing when using CLI
e5d0ad4 
* Feature: Help message;
* Feature: Directory change (-C);
* Feature #7: Runnerfile override (-f);
* Fix: Fixed sourcing bug in index.sh;
* Improvement: Use runner_{args,tasks,flags} are now arrays instead of strings;
* Improvement: Quote all unquoted variables to minimize risk of splitting arguments where not needed.
@stylemistake
Copy link
Owner Author

stylemistake commented May 24, 2016 •
edited
Loading

@mbrodala I summon you to test this shit out. :)

I also cleaned up the pavement for you to implement the bash completion.

@mbrodala
Copy link
Collaborator

I'll have a look. :-)

@mbrodala
Copy link
Collaborator

mbrodala commented May 25, 2016 •
edited
Loading

How is one supposed to execute the runner on CLI now? You've recently dropped the bin/runner file.

Update: got it working by running index.sh instead but that feels odd. ;-)

@mbrodala
Copy link
Collaborator

Overriding the runnerfile seems to work fine:

$ cat > foo.sh
task_foo() {
  echo "Foo"
}
$ bash bash-task-runner/index.sh -f foo.sh foo
[10:01:01.472] Starting 'foo'...
Foo
[10:01:01.478] Finished 'foo' after 2 ms

@stylemistake
Copy link
Owner Author

Ok, dropping bin/runner was a bad idea. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cli feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stylemistake @mbrodala