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 action versions #287

Merged
merged 1 commit into from
Aug 24, 2024
Merged

Conversation

diegodlh
Copy link
Owner

The test job in the release workflow was throwing some warnings because the actions it used were using deprecated versions of Node. See for example here: https://github.com/diegodlh/zotero-cita/actions/runs/10478095223

I understand that using newer versions of the actions used should fix it. But because I've never used Github workflows or actions before, I wanted to make sure before merging the changes.

@Dominic-DallOsto, could you please review? Thanks!

The `test` job in the `release` workflow was throwing some warnings because the actions it used were using deprecated versions of Node. See for example here: https://github.com/diegodlh/zotero-cita/actions/runs/10478095223

I understand that using newer versions of the actions used should fix it. But because I've never used Github workflows or actions before, I wanted to make sure before merging the changes.

@Dominic-DallOsto, could you please review? Thanks!
@diegodlh
Copy link
Owner Author

Just noting that the test job has run successfully with the changes made and no warnings were thrown: https://github.com/diegodlh/zotero-cita/actions/runs/10514593683

I guess it should be safe to merge :)

@Dominic-DallOsto Dominic-DallOsto merged commit 365b12b into zotero7 Aug 24, 2024
1 check passed
@Dominic-DallOsto Dominic-DallOsto deleted the release-workflow-action-versions branch August 24, 2024 10:40
Copy link

🚀 This ticket has been resolved in v1.0.0-beta.0. See Release 1.0.0-beta.0 for release notes.

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.

2 participants