Skip to content

Conversation

@andponlin
Copy link
Contributor

If the project does not have the avaje-inject dependency configured on the project but the code-generator is configured then it will yield an NPE. This fix will at least raise an exception with an explanation.

If the project does not have the `avaje-inject` dependency
configured on the project but the code-generator is configured
then it will yield an NPE. This fix will at least raise an
exception with an explanation.
@andponlin andponlin force-pushed the andponlin-error-for-missing-dep branch from a7c7dfe to 9881419 Compare November 9, 2025 08:15
Copy link
Collaborator

@SentryMan SentryMan left a comment

Choose a reason for hiding this comment

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

One wonders how you got into this situation but the change looks good to me

@SentryMan SentryMan enabled auto-merge (squash) November 9, 2025 13:23
@SentryMan SentryMan merged commit 1397575 into avaje:master Nov 9, 2025
5 checks passed
@SentryMan SentryMan added the bug Something isn't working label Nov 9, 2025
@SentryMan SentryMan added this to the 12.1 milestone Nov 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants