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

Do not use deprecated set-output command #9

Closed
wants to merge 1 commit into from
Closed

Do not use deprecated set-output command #9

wants to merge 1 commit into from

Conversation

giordano
Copy link

@vsoch
Copy link
Owner

vsoch commented Jan 12, 2023

Yep these changes look great! And I'm definitely aware - it was such a common task I made a tool for handling it: https://github.com/vsoch/action-updater.

If you like this prototype action, you might also be interested in:

@vsoch
Copy link
Owner

vsoch commented Jan 12, 2023

Looks like it can't find spec_jsons in the buildcache test https://github.com/vsoch/spack-package-action/actions/runs/3902914005/workflow#L34-L38 but I couldn't tell you why.

@giordano
Copy link
Author

If I'm reading https://github.com/vsoch/spack-package-action/actions/runs/3902914005/jobs/6671759103#step:3:214 correctly, that's the output of

# Did we make stuff?
tree ${build_cache}
but there are no *.json files and so the loop
for spec_json in $(find ${build_cache} -name *.json); do
is empty.

@giordano giordano closed this May 28, 2023
@giordano giordano deleted the mg/de-set-output branch May 28, 2023 22:57
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.

None yet

2 participants