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

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

@github-actions github-actions bot added the stale label Aug 9, 2020
@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 •
edited
Loading

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.

@epignatelli epignatelli mentioned this issue Sep 13, 2022
Closed
@JackPGreen JackPGreen mentioned this issue May 1, 2024
Merged
mergify bot added a commit that referenced this issue May 13, 2024
@JackPGreen @mergify
Add documentation for dryrun parameter (#2311)
070c257 
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
@JackPGreen @mergify @chhe
Add documentation for dryrun parameter (#2311)
8ffcddb 
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
@JackPGreen @mergify @chhe
chore: Add documentation for dryrun parameter (#2311)
01bd249 
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 22, 2024
@JackPGreen @mergify @chhe
chore: Add documentation for dryrun parameter (#2311)
9f69cd2 
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 Jul 30, 2024
@JackPGreen @mergify @chhe
chore: Add documentation for dryrun parameter (#2311)
9ffbef4 
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 Sep 26, 2024
@JackPGreen @mergify @chhe
chore: Add documentation for dryrun parameter (#2311)
491e264 
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 Oct 6, 2024
@JackPGreen @mergify @chhe
chore: Add documentation for dryrun parameter (#2311)
b876342 
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 Nov 1, 2024
@JackPGreen @mergify @chhe
Add documentation for dryrun parameter (#2311)
e78aa4f 
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 Nov 26, 2024
@JackPGreen @mergify @chhe
Add documentation for dryrun parameter (#2311)
b0719f1 
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
Assignees
No one assigned
Labels
needs-work Extra attention is needed stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add documentation for dryrun parameter
4 participants
@Pomax @cplee @hellmrf @catthehacker