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

Mention --assume-ready of datalad-run #699

Merged
merged 5 commits into from
May 17, 2021
Merged

Mention --assume-ready of datalad-run #699

merged 5 commits into from
May 17, 2021

Conversation

adswa
Copy link
Contributor

@adswa adswa commented Apr 9, 2021

This adds a short paragraph in the chapter on datalad run's input and output options,
and adds a bullet point in the summary of datalad run, on the recently introduced
--assume-ready flag that skips the preparation of --inputs and --outputs if a user
has retrieved/unlocked them already
Fixes #697.

ping @mih

This adds a short paragraph in the chapter on datalad run's input and output options,
and adds a bullet point in the summary of datalad run, on the recently introduced
--assume-ready flag that skips the preparation of --inputs and --outputs if a user
has retrieved/unlocked them already
Fixes #697.
@adswa
Copy link
Contributor Author

adswa commented Apr 10, 2021

Mhh, the unit tests seem to be messed up. Pushing an empty commit to retrigger the build

@adswa
Copy link
Contributor Author

adswa commented Apr 10, 2021

Somehow we lost travis. Investigating...

@adswa adswa force-pushed the assumeready branch 8 times, most recently from 8f715d3 to 04e4256 Compare April 10, 2021 11:14
@adswa
Copy link
Contributor Author

adswa commented Apr 10, 2021

So, we ran out of Travis.com credits. For some reason, I thought they allocated a fixed but recurring amount of credits to open source projects, but its a one-time allocation only. I have now replaced the travis setup with github actions. They provide 2000 build minutes per month for free, which should suffice.

Replace Travis setup with github actions

We ran out of credits, and will try to switch to github actions' 2000
build minutes per month
Reduce the amount of builds - one build per push is enough
@adswa
Copy link
Contributor Author

adswa commented Apr 23, 2021

ping @mih - is this good to go?

@adswa
Copy link
Contributor Author

adswa commented Apr 23, 2021

oh dang, this branch has the CI setup...

Copy link
Collaborator

@mih mih left a comment

Choose a reason for hiding this comment

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

Yes, this works. Thx!

@mih mih merged commit c593f36 into master May 17, 2021
@mih mih deleted the assumeready branch May 17, 2021 06:51
@adswa adswa mentioned this pull request May 25, 2021
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.

Mention run --assume-ready
2 participants