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

Move plugins to RefreshVersions #5016

Merged
merged 3 commits into from
Sep 17, 2024

Conversation

mikescamell
Copy link
Contributor

@mikescamell mikescamell commented Sep 16, 2024

Task/Issue URL: https://app.asana.com/0/1207908166761516/1208315099637999/f

Description

Moves our plugins to refreshVersions so that they will be updated when the refreshVersions task is run.

As part of this moved all but the AGP plugin to the plugins block. Not quite sure why that one didn’t want to work but don’t want to spend too much time on it, it’s still part of RefreshVersions.

I did not update the plugins as part of this PR.

Steps to test this PR

Feature 1

  • Check the plugin versions in versions.properties match the previous versions
  • Run ./gradlew refreshVersions and check the plugins get update comments (if they have an update of course)
  • Sync, build & run should be successful

UI changes

N/A

Screenshot of the changes working

Screenshot 2024-09-17 at 11 19 50

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mikescamell and the rest of your teammates on Graphite Graphite

@mikescamell
Copy link
Contributor Author

Needs to be updated and checked when #4971 gets merged.

@malmstein malmstein self-assigned this Sep 16, 2024
@mikescamell mikescamell force-pushed the feature/mike/refreshversions-plugins branch 2 times, most recently from 56a0273 to 58df0e4 Compare September 17, 2024 09:13
we're taking the opportunity to put the plugins in the plugin block (the new recommended way of declaring plugins)

I had problems with AGP so left it of for now, we still use refreshVersions for it
now we have the anvil plugin in the versions.properties file we want to make sure that it, and the anvil dependencies stay on the same version so we fetch the anvil version declared in the version.properties file
we also ditch the version declaration in other gradle files as they are no longer needed
@mikescamell mikescamell force-pushed the feature/mike/refreshversions-plugins branch from 58df0e4 to 4015f1e Compare September 17, 2024 09:18
@mikescamell mikescamell marked this pull request as ready for review September 17, 2024 10:00
Copy link
Contributor

@malmstein malmstein left a comment

Choose a reason for hiding this comment

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

nice work 🚢 it!

@mikescamell mikescamell merged commit 5a70f8b into develop Sep 17, 2024
7 checks passed
@mikescamell mikescamell deleted the feature/mike/refreshversions-plugins branch September 17, 2024 15:14
mikescamell added a commit that referenced this pull request Sep 17, 2024
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