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

Update specific inputs #14

Closed
a-kenji opened this issue Nov 29, 2021 · 7 comments · Fixed by #15
Closed

Update specific inputs #14

a-kenji opened this issue Nov 29, 2021 · 7 comments · Fixed by #15

Comments

@a-kenji
Copy link
Contributor

a-kenji commented Nov 29, 2021

Thank you for this project!
Is it possible to have this action update specific inputs?

@cole-h
Copy link
Member

cole-h commented Dec 1, 2021

Thanks for the report! #15 should support this -- I'll cut a new release later.

@a-kenji
Copy link
Contributor Author

a-kenji commented Dec 2, 2021

Thank you!

@cole-h
Copy link
Member

cole-h commented Dec 2, 2021

I just released https://github.com/DeterminateSystems/update-flake-lock/releases/tag/v5 -- thanks again!

@a-kenji
Copy link
Contributor Author

a-kenji commented Dec 2, 2021

Thank you, it works great!

But it seems that I need to add the script now to the root of my project.
Is that expected?

/home/runner/work/_temp/dd767b88-96e4-4375-8ba5-bd14e9de88d3.sh: line 1: ./update-input-or-inputs.sh: No such file or directory

If I add the script, then the action proceeds without problems.

@cole-h
Copy link
Member

cole-h commented Dec 2, 2021

Not expected indeed. I'll see if I can't figure something out.

@cole-h
Copy link
Member

cole-h commented Dec 2, 2021

I've opened #18 -- unfortunately, I won't be able to take a look until tomorrow. In the meantime, if you use DeterminateSystems/update-flake-lock@0d0deff8bd5e1859adc80318e918e5e90bd71d80, that should give you what you want (which is the commit before I broke the script out into its own file).

@a-kenji
Copy link
Contributor Author

a-kenji commented Dec 2, 2021

Awesome, thank you.
And no need to rush!

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 a pull request may close this issue.

2 participants