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

Address CI Action EOL #503

Merged
merged 4 commits into from
Jan 31, 2025
Merged

Address CI Action EOL #503

merged 4 commits into from
Jan 31, 2025

Conversation

vtnate
Copy link
Collaborator

@vtnate vtnate commented Jan 31, 2025

Resolves #[issue number here]

Pull Request Description

Upload-artifact v3 reached its end of life on 2025-01-30. To continue saving CI output from failed runs (for a week, as before), we are required to use v4 of that Action, and update our naming technique.

And then, since I was messing around in the .github folder, I updated our templates. Easy to revert if we don't like those changes.

Checklist (Delete lines that don't apply)

  • All ci tests pass (green)
  • This branch is up-to-date with develop

@vtnate vtnate added the Tests This is entirely within our test suite label Jan 31, 2025
@vtnate vtnate self-assigned this Jan 31, 2025
# Only upload if a previous step fails
if: failure()
with:
name: rspec_results
name: rspec_results_${{ matrix.simulation-type }}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is the key change required by the upload-artifact version bump

@@ -8,5 +8,5 @@

- [ ] Unit tests have been added or updated
- [ ] All ci tests pass (green)
- [ ] An [issue](https://github.com/urbanopt/urbanopt-cli/issues) has been created (which will be used for the changelog)
- [ ] This PR has been labeled appropriately (which will be used for the changelog)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Different reminder for the current technique of building our changelog

@vtnate vtnate requested a review from kflemin January 31, 2025 16:26
@vtnate vtnate marked this pull request as ready for review January 31, 2025 16:26
@vtnate vtnate merged commit ed17237 into develop Jan 31, 2025
5 checks passed
@vtnate vtnate deleted the fix-ci branch January 31, 2025 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tests This is entirely within our test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants