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.

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

Added ability to pass options to scripts #138

Merged
merged 1 commit into from
May 17, 2017

Conversation

kwelch
Copy link
Contributor

@kwelch kwelch commented May 17, 2017

What:
Added ability to pass options to scripts

Why:
to address #133

How:
Appended options from scriptNames array to the last script prior to --

@codecov-io
Copy link

Codecov Report

Merging #138 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #138   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          10     10           
  Lines         294    302    +8     
=====================================
+ Hits          294    302    +8
Impacted Files Coverage Δ
src/index.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f95d207...c50d3f9. Read the comment docs.

Copy link
Collaborator

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect!

@kentcdodds kentcdodds merged commit 061b5c3 into sezna:master May 17, 2017
@kentcdodds
Copy link
Collaborator

This is great. Thank you!

@dkent600
Copy link

dkent600 commented Jun 6, 2018

Looking forward to being able to use this!

@dkent600
Copy link

dkent600 commented Jun 6, 2018

Could use some documentation here on how to use

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants