-
Notifications
You must be signed in to change notification settings - Fork 738
Dry run #4214
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
Dry run #4214
Conversation
✅ Deploy Preview for nextflow-docs-staging canceled.
|
This comment was marked as off-topic.
This comment was marked as off-topic.
fd99141
to
19d2ccb
Compare
4e27468
to
dfd7d09
Compare
✅ Deploy Preview for nextflow-docs-staging ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This comment was marked as outdated.
This comment was marked as outdated.
- I would consider enabling this "deep" cache analysis only when the dry run is enabled
And / or as a opt-in? I can imagine some people (me) wanting it routinely if not doing large runs. Might be annoying to have to go back and re-do a dry run every time.
1f834a0
to
6454605
Compare
5a93547
to
27345a6
Compare
f6a3696
to
49b58d2
Compare
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
980747f
to
9e8b1f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check meaning is retained.
b4b321e
to
069653d
Compare
b7b4221
to
c1114bc
Compare
Close #844