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

build(deps): bump com.google.dagger:dagger-compiler from 2.47 to 2.48.1 #1693

Conversation

dependabot[bot]
Copy link
Contributor

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

Bumps com.google.dagger:dagger-compiler from 2.47 to 2.48.1.

Release notes

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

Dagger 2.48.1

Bug Fixes

Fixes #4063, #4054: [KSP]: Fixes some incremental processing issues with KSP (c8a568956) Fixes #4046: [KSP] Fixes "Unexpected element" in XProcessing library (androidx/androidx@bdc1bb0) Fixes #4059: [KSP] Align behavior of generic inline types in KSP with KAPT in XProcessing library (androidx/androidx@b0fa9cf) Fixes #3980: Fixes Lint crash with AGP 8.1.0. (e651294c2) Fixes #4055: Fixes issue with using generic types in Dagger's ClassKey (9852b4237)

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:

If you depend on androidx.hilt:hilt-common or androidx.hilt:hilt-work they will need to be updated to at least 1.1.0-alpha01.

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
</tr></table> 

... (truncated)

Commits
  • 2d9ba60 2.48.1 release
  • 378535e Refactor ResolvedBindings and LegacyBindingGraph to include the ComponentPath.
  • fd907cb Allow sharding for Dagger compiler tests.
  • 558cc51 Fix Dagger's incremental processing for KSP.
  • 5cc209c [Refactor]: This CL refactors a few things in ComponentDescriptor.
  • e8f5f2b Drop latest XProcessing jars into Dagger.
  • 7412301 Internal changes
  • 76bde06 Rollback of "[Refactor] Remove duplicated logic in BindingGraphFactory"
  • e651294 Fix Lint crash with AGP 8.1.0. See google/dagger#4023...
  • 47953aa [Refactor] Remove duplicated logic in BindingGraphFactory.
  • Additional commits viewable in compare view

Dependabot compatibility score

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 chore Refactor, rename, cleanup, etc. dependencies Pull requests that update a dependency file safe-to-test labels Oct 4, 2023
@dependabot dependabot bot requested a review from a team October 4, 2023 00:24
@github-actions
Copy link
Contributor

github-actions bot commented Oct 4, 2023

Hi @dependabot[bot]! Add at least one of the required labels to this PR

Required labels are : chore,ci,cleanup,docs,feat,fix,perf,refactor,style,test

@github-actions github-actions bot added the needs-triage Needs thorough attention from code reviewers label Oct 4, 2023
@dependabot dependabot bot force-pushed the dependabot/maven/com.google.dagger-dagger-compiler-2.48.1 branch from 264b30c to 8b6966e Compare October 4, 2023 18:10
@dependabot dependabot bot force-pushed the dependabot/maven/com.google.dagger-dagger-compiler-2.48.1 branch 12 times, most recently from c7aa9c0 to 90df50e Compare October 17, 2023 14:12
@dependabot dependabot bot force-pushed the dependabot/maven/com.google.dagger-dagger-compiler-2.48.1 branch from 90df50e to 9fcc411 Compare October 24, 2023 14:04
@dependabot dependabot bot requested a review from a team as a code owner October 24, 2023 14:04
@dependabot dependabot bot force-pushed the dependabot/maven/com.google.dagger-dagger-compiler-2.48.1 branch 4 times, most recently from 023d05c to 2232dea Compare October 26, 2023 15:35
Bumps [com.google.dagger:dagger-compiler](https://github.com/google/dagger) from 2.47 to 2.48.1.
- [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.1)

---
updated-dependencies:
- dependency-name: com.google.dagger:dagger-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/maven/com.google.dagger-dagger-compiler-2.48.1 branch from 2232dea to 086aea8 Compare October 27, 2023 15:20
Copy link
Contributor Author

dependabot bot commented on behalf of github Dec 4, 2023

Superseded by #1807.

@dependabot dependabot bot closed this Dec 4, 2023
@dependabot dependabot bot deleted the dependabot/maven/com.google.dagger-dagger-compiler-2.48.1 branch December 4, 2023 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Refactor, rename, cleanup, etc. dependencies Pull requests that update a dependency file needs-triage Needs thorough attention from code reviewers safe-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants