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

[Tooling] Update GitHub Actions #1718

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

iangmaia
Copy link
Contributor

Reference: p7H4VZ-59z-p2

Simplenote Android doesn't use one of the GitHub Actions about to have its older versions deprecated as actions/upload-artifact, actions/download-artifact or actions/cache, but I used this opportunity to upgrade the Action gradle/actions/wrapper-validation and to move the Buildkite Gradle Wrapper Validation to the Linter queue.

@iangmaia iangmaia added the tooling Related to anything that supports the building & maintaining of the project. label Dec 12, 2024
@iangmaia iangmaia added this to the Future milestone Dec 12, 2024
@iangmaia iangmaia self-assigned this Dec 12, 2024
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Dec 12, 2024

📲 You can test the changes from this Pull Request in Simplenote Android by scanning the QR code below to install the corresponding build.

App Name Simplenote Android
Build TypeDebug
Commita3a00bd
Direct Downloadsimplenote-android-prototype-build-pr1718-a3a00bd-0193bfce-14fb-45cf-b3c5-01e5d60acb95.apk

@iangmaia iangmaia force-pushed the iangmaia/update-gradle-wrapper-validation branch from f0820b4 to cdb6b71 Compare December 12, 2024 17:09
@iangmaia iangmaia requested a review from a team December 12, 2024 17:14
@spencertransier
Copy link
Contributor

@iangmaia Approved to unblock, but is there a reason this change would affect the linting errors that Danger picked up?

@iangmaia
Copy link
Contributor Author

@iangmaia Approved to unblock, but is there a reason this change would affect the linting errors that Danger picked up?

It's not related to these changes but they might have been dormant for some reason -- my guess is that the Linter didn't pick them up in a previous PR due to a bug (now fixed) that it would use a different Rubocop version than the one configured in the project Bundle.
I confirmed locally that Rubocop reports the same issues, so I fixed them and pushed on a3a00bd.

@iangmaia iangmaia merged commit c946759 into trunk Dec 13, 2024
15 checks passed
@iangmaia iangmaia deleted the iangmaia/update-gradle-wrapper-validation branch December 13, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tooling Related to anything that supports the building & maintaining of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants