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

Document what "dryrun" means for this tool? #268

Closed
Pomax opened this issue Jun 7, 2020 · 4 comments · Fixed by #2311
Closed

Document what "dryrun" means for this tool? #268

Pomax opened this issue Jun 7, 2020 · 4 comments · Fixed by #2311
Labels
needs-work Extra attention is needed stale

Comments

@Pomax
Copy link

Pomax commented Jun 7, 2020

Dry running can mean many different things, can the readme and help output be updated to explain what dryrun means for this tool? (e.g. do things like git push work? if not, which things don't work, etc)

Especially for actions that send resultant data off for hosting/further external processing, it's pretty important to know whether or not there is a way to prevent act from allowing that to happen.

@cplee cplee added the needs-work Extra attention is needed label Jun 9, 2020
@github-actions
Copy link
Contributor

github-actions bot commented Aug 9, 2020

Issue is stale and will be closed in 7 days unless there is new activity

@hellmrf
Copy link

hellmrf commented Feb 28, 2022

This is still a problem. What dryrun means? I just get a bunch of errors...

@catthehacker
Copy link
Member

Dry run will not create any containers so it will validate only workflow correctness.

@Pomax
Copy link
Author

Pomax commented Mar 2, 2022

If that is the true and correct information, I would like to reiterate my request to update the README.md and act --help output with that information, to prevent future uncertainty.

JackPGreen added a commit to JackPGreen/act that referenced this issue May 1, 2024
The `dryrun` paramater's documentation is unhelpful (`dryrun mode`).

In the absense of proper documentation, [this definition from a comment on this topic](nektos#268 (comment)) should be used.

Fixes: nektos#268
mergify bot added a commit that referenced this issue May 13, 2024
The `dryrun` paramater's documentation is unhelpful (`dryrun mode`).

In the absense of proper documentation, [this definition from a comment on this topic](#268 (comment)) should be used.

Fixes: #268

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
chhe pushed a commit to chhe/act that referenced this issue May 14, 2024
The `dryrun` paramater's documentation is unhelpful (`dryrun mode`).

In the absense of proper documentation, [this definition from a comment on this topic](nektos/act#268 (comment)) should be used.

Fixes: nektos/act#268

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
chhe pushed a commit to chhe/act that referenced this issue May 14, 2024
The `dryrun` paramater's documentation is unhelpful (`dryrun mode`).

In the absense of proper documentation, [this definition from a comment on this topic](nektos/act#268 (comment)) should be used.

Fixes: nektos/act#268

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-work Extra attention is needed stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants