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

fix(deps): Bump hilt_version from 2.47 to 2.48 #37

Closed
wants to merge 1 commit into from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 4, 2023

Bumps hilt_version from 2.47 to 2.48.
Updates com.google.dagger:hilt-android from 2.47 to 2.48

Release notes

Sourced from com.google.dagger:hilt-android's releases.

Dagger 2.48

Dagger/Hilt KSP support

Dagger 2.48 includes the alpha release of the Dagger and Hilt KSP processors.

Instructions for using the Dagger/Hilt KSP processors can be found at https://dagger.dev/dev-guide/ksp.

In order to use Dagger’s KSP processor you will need to:

Also note that Dagger’s KSP processors are still in the alpha stage. So far we’ve focused mainly on trying to ensure correctness rather than optimize performance. Please apply due diligence when enabling ksp and report any bugs or performance issues at https://github.com/google/dagger/issues. The current list of known issues can be found here.

There are also a few potentially breaking changes included with this release. These changes were made to better support Dagger usage with Kotlin sources, and make the migration from KAPT to KSP more seamless. We don’t expect these changes to affect most users. Please see below for more details.

Breaking changes

The dagger.ignoreProvisionKeyWildcards is now enabled by default

This may break apps that are providing the same binding with different wildcards, e.g. Foo<Bar> and Foo<? extends Bar>.

Fix: See https://dagger.dev/dev-guide/compiler-options#ignore-provision-key-wildcards for suggestions on how to fix this. If fixing is not an immediate option, you can still disable the flag with dagger.ignoreProvisionKeyWildcards=DISABLED.

@Binds assignability check

Unlike KAPT, KSP takes nullability into account when checking if a type is assignable to another type. This changes the behavior of Dagger’s @Binds usage validation such that a type that was assignable in KAPT may no longer be assignable in KSP. For example:

// Incorrect: this compiles successfully in KAPT but the compilation fails in KSP
@Binds fun bind(impl: FooImpl<Bar?>): Foo<Bar>

Fix: To fix this breakage, users should update the parameter or return type in the method such that they are actually assignable when taking nullability into account. For example:

// Correct: this compiles successfully in KAPT and KSP
@Binds fun bind(impl: FooImpl<Bar?>): Foo<Bar?>

Top-level @Module / @Inject classes can no longer be private

Top-level @Module or @Inject classes can no longer be private. This was previously allowed purely by accident (rather than being an officially supported feature) due to the way Kotlin’s private classes are represented as package-private in KAPT’s generated java stubs.

Fix: Replace the private visibility modifier with internal or public.

@Binds and @Provides methods can no longer be extension functions

Normally when we define an @Provides/@Binds function we put the dependencies in the parameter list:

... (truncated)

Commits
  • 03513e5 2.48 release
  • b90b416 Don't run RootProcessingStep#postProcess if no root element was detected.
  • f5d686d Ban usage of var in Component.
  • 232e524 Fix github action
  • 046baa4 Remove workaround for b/269172737.
  • 7107431 Rename #java() to #javac() for Dagger SPI model types.
  • b7d97ba Internal change
  • 278e4ec Update xprocessing jar
  • c53be9a UPDATE Bazel version to 6.3.2
  • 8d2385f Move Dagger spi model types implementation into Dagger internal.
  • Additional commits viewable in compare view

Updates com.google.dagger:hilt-compiler from 2.47 to 2.48

Release notes

Sourced from com.google.dagger:hilt-compiler's releases.

Dagger 2.48

Dagger/Hilt KSP support

Dagger 2.48 includes the alpha release of the Dagger and Hilt KSP processors.

Instructions for using the Dagger/Hilt KSP processors can be found at https://dagger.dev/dev-guide/ksp.

In order to use Dagger’s KSP processor you will need to:

Also note that Dagger’s KSP processors are still in the alpha stage. So far we’ve focused mainly on trying to ensure correctness rather than optimize performance. Please apply due diligence when enabling ksp and report any bugs or performance issues at https://github.com/google/dagger/issues. The current list of known issues can be found here.

There are also a few potentially breaking changes included with this release. These changes were made to better support Dagger usage with Kotlin sources, and make the migration from KAPT to KSP more seamless. We don’t expect these changes to affect most users. Please see below for more details.

Breaking changes

The dagger.ignoreProvisionKeyWildcards is now enabled by default

This may break apps that are providing the same binding with different wildcards, e.g. Foo<Bar> and Foo<? extends Bar>.

Fix: See https://dagger.dev/dev-guide/compiler-options#ignore-provision-key-wildcards for suggestions on how to fix this. If fixing is not an immediate option, you can still disable the flag with dagger.ignoreProvisionKeyWildcards=DISABLED.

@Binds assignability check

Unlike KAPT, KSP takes nullability into account when checking if a type is assignable to another type. This changes the behavior of Dagger’s @Binds usage validation such that a type that was assignable in KAPT may no longer be assignable in KSP. For example:

// Incorrect: this compiles successfully in KAPT but the compilation fails in KSP
@Binds fun bind(impl: FooImpl<Bar?>): Foo<Bar>

Fix: To fix this breakage, users should update the parameter or return type in the method such that they are actually assignable when taking nullability into account. For example:

// Correct: this compiles successfully in KAPT and KSP
@Binds fun bind(impl: FooImpl<Bar?>): Foo<Bar?>

Top-level @Module / @Inject classes can no longer be private

Top-level @Module or @Inject classes can no longer be private. This was previously allowed purely by accident (rather than being an officially supported feature) due to the way Kotlin’s private classes are represented as package-private in KAPT’s generated java stubs.

Fix: Replace the private visibility modifier with internal or public.

@Binds and @Provides methods can no longer be extension functions

Normally when we define an @Provides/@Binds function we put the dependencies in the parameter list:

... (truncated)

Commits
  • 03513e5 2.48 release
  • b90b416 Don't run RootProcessingStep#postProcess if no root element was detected.
  • f5d686d Ban usage of var in Component.
  • 232e524 Fix github action
  • 046baa4 Remove workaround for b/269172737.
  • 7107431 Rename #java() to #javac() for Dagger SPI model types.
  • b7d97ba Internal change
  • 278e4ec Update xprocessing jar
  • c53be9a UPDATE Bazel version to 6.3.2
  • 8d2385f Move Dagger spi model types implementation into Dagger internal.
  • Additional commits viewable in compare view

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Sep 4, 2023
kodiakhq[bot]
kodiakhq bot previously approved these changes Sep 4, 2023
@github-actions github-actions bot added the automerge Allow kodiak to automerge commit when all checks pass label Sep 4, 2023
@dependabot dependabot bot force-pushed the dependabot/gradle/hilt_version-2.48 branch from bf30f29 to efd9537 Compare September 4, 2023 12:31
@dependabot dependabot bot force-pushed the dependabot/gradle/hilt_version-2.48 branch from efd9537 to 40f24cd Compare September 11, 2023 11:09
kodiakhq[bot]
kodiakhq bot previously approved these changes Sep 11, 2023
@kodiakhq kodiakhq bot removed the automerge Allow kodiak to automerge commit when all checks pass label Sep 15, 2023
@kodiakhq
Copy link

kodiakhq bot commented Sep 15, 2023

This PR currently has a merge conflict. Please resolve this and then re-add the automerge label.

@dependabot dependabot bot force-pushed the dependabot/gradle/hilt_version-2.48 branch from 40f24cd to 5e8ee92 Compare September 15, 2023 17:18
@dependabot dependabot bot force-pushed the dependabot/gradle/hilt_version-2.48 branch 5 times, most recently from 50e6e9d to d620c5e Compare September 24, 2023 13:10
@dependabot dependabot bot force-pushed the dependabot/gradle/hilt_version-2.48 branch 4 times, most recently from e0521c5 to aaa4729 Compare September 27, 2023 13:10
@dependabot dependabot bot force-pushed the dependabot/gradle/hilt_version-2.48 branch 2 times, most recently from cb06cac to 86b910f Compare October 2, 2023 12:43
Bumps `hilt_version` from 2.47 to 2.48.

Updates `com.google.dagger:hilt-android` from 2.47 to 2.48
- [Release notes](https://github.com/google/dagger/releases)
- [Changelog](https://github.com/google/dagger/blob/master/CHANGELOG.md)
- [Commits](google/dagger@dagger-2.47...dagger-2.48)

Updates `com.google.dagger:hilt-compiler` from 2.47 to 2.48
- [Release notes](https://github.com/google/dagger/releases)
- [Changelog](https://github.com/google/dagger/blob/master/CHANGELOG.md)
- [Commits](google/dagger@dagger-2.47...dagger-2.48)

---
updated-dependencies:
- dependency-name: com.google.dagger:hilt-android
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.google.dagger:hilt-compiler
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/gradle/hilt_version-2.48 branch from 86b910f to 78e255c Compare October 2, 2023 21:17
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Oct 9, 2023

Superseded by #103.

@dependabot dependabot bot closed this Oct 9, 2023
@dependabot dependabot bot deleted the dependabot/gradle/hilt_version-2.48 branch October 9, 2023 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants