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

Add support to prevent execution #104

Merged
merged 2 commits into from
Aug 2, 2018
Merged

Add support to prevent execution #104

merged 2 commits into from
Aug 2, 2018

Conversation

smarr
Copy link
Owner

@smarr smarr commented Aug 2, 2018

This allows us to do a dry-run.
It processes all other arguments normally, and loads the configuration.
This ensures that the configuration matches the schema.
And perhaps also other basic properties.
Though, it isn't a total guarantee that the configuration is going to work.

This is essentially a dry-run, and allows to check whether the loads configuration.

Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr added the Feature label Aug 2, 2018
@smarr smarr added this to the v1.0 - Foundation milestone Aug 2, 2018
@smarr smarr self-assigned this Aug 2, 2018
@coveralls
Copy link

Coverage Status

Coverage increased (+1.6%) to 88.509% when pulling 4611f38 on no-exec into 43a1d15 on master.

@coveralls
Copy link

coveralls commented Aug 2, 2018

Coverage Status

Coverage decreased (-0.4%) to 86.443% when pulling c51f968 on no-exec into 43a1d15 on master.

@smarr smarr force-pushed the no-exec branch 6 times, most recently from dba462d to d24ba99 Compare August 2, 2018 20:26
There are currently issues with PyPy for some reason.

Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr merged commit 2423a1b into master Aug 2, 2018
@smarr smarr deleted the no-exec branch August 2, 2018 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants