Skip to content

Conversation

@pkoenig10
Copy link
Member

@pkoenig10 pkoenig10 commented Jun 10, 2025

Another attempt at #3101.

'''.stripIndent()
}

def 'compileJava applies patches when errorProneApply contains specific checks including disabled'() {
Copy link
Member Author

@pkoenig10 pkoenig10 Jun 10, 2025

Choose a reason for hiding this comment

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

This is an Error Prone behavior change. See google/error-prone#4699 (comment).

I'm not sure there is much we can do about this. I'm not sure if or how heavily we are relying on this behavior in our internal automations. It appears that we deliberately wanted this behavior in #1109, but then a lot of this logic was removed in #2944.

@pkoenig10 pkoenig10 requested a review from FinlayRJW June 10, 2025 13:21
Copy link
Contributor

@CRogers CRogers left a comment

Choose a reason for hiding this comment

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

Thanks!

@bulldozer-bot bulldozer-bot bot merged commit ccb8b20 into develop Jun 11, 2025
5 checks passed
@bulldozer-bot bulldozer-bot bot deleted the pkoenig/ep2 branch June 11, 2025 16:29
@autorelease3
Copy link

autorelease3 bot commented Jun 11, 2025

Released 6.33.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants