Skip to content

fix: updated dependencies to solve compilation issue #1557

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

Merged
merged 6 commits into from
May 2, 2024

Conversation

kikoso
Copy link
Collaborator

@kikoso kikoso commented May 1, 2024

This PR fixes the issue #1554 and enables again the compilation of all samples.

Like it often happens, the initial idea was to simply upgrade the compileSdk version to make it work again.

(narrator voice: it wasn't that simple)

This required:

  • Upgrading the AGP.
  • Updated the Manifest in the WearOS project
  • Since the Manifest was modified, updating the header
  • Added the Kotlin BoM to prevent class duplication.

This update should give some peace of mind to the android-samples project.


Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #1554 🦕

@kikoso kikoso requested review from wangela and dkhawk May 1, 2024 13:25
@kikoso kikoso self-assigned this May 1, 2024
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
#Wed May 01 14:29:09 CEST 2024
distributionBase=GRADLE_USER_HOME
Copy link
Contributor

Choose a reason for hiding this comment

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

Extra carriage returns here?

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 automatically added when using ./gradlew wrapper to update the Gradle version. We can probably leave it as it is.

@dkhawk dkhawk merged commit 1df7d3d into main May 2, 2024
11 checks passed
@dkhawk dkhawk deleted the fix/updated_dependencies_to_solve_compilation_issue branch May 2, 2024 20:54
@googlemaps-bot
Copy link
Contributor

🎉 This PR is included in version 1.10.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@googlemaps-samples googlemaps-samples deleted a comment from cklim5 May 10, 2024
kikoso added a commit that referenced this pull request Aug 14, 2024
* fix: versions and dependencies updated to allow compilation

* fix: duplicated libraries

* fix: updated Gradle

* fix: updated Manifest

* fix: updated Gradle

* fix: fixed header
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update dependencies so that tests pass
3 participants