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 remaining GPS packages to AndroidX repository. #955

Merged
merged 1 commit into from
Sep 13, 2024
Merged

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Sep 4, 2024

"Pause" migrating everything to a common template, and pivot to moving the remaining packages from the GPS repository to the AndroidX repository. This should enable us to eliminate the circular dependencies between the two repositories which should help fix the majority of the #764 issues going forward.

Bring in the GPS and Glide templates as new "templateSet", and merge in all the build infrastructure pieces that the GPS repository had to make it build correctly.

TODO:

@jpobst jpobst force-pushed the gps-phase-3 branch 15 times, most recently from 602976c to e8c842b Compare September 6, 2024 21:48
@jpobst jpobst changed the title Move third batch of GPS packages to AndroidX repository. Move remaining GPS packages to AndroidX repository. Sep 6, 2024
@jpobst jpobst force-pushed the gps-phase-3 branch 3 times, most recently from ed5899c to 5a58044 Compare September 12, 2024 20:11
@jpobst jpobst marked this pull request as ready for review September 13, 2024 00:00
@jpobst jpobst requested a review from moljac September 13, 2024 00:00
Copy link
Contributor

@moljac moljac left a comment

Choose a reason for hiding this comment

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

LGTM

I hope I did not miss something

@jpobst jpobst merged commit 8d465e8 into main Sep 13, 2024
26 checks passed
@jpobst jpobst deleted the gps-phase-3 branch September 13, 2024 17:22
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