Skip to content

Disable Error Prone's StringConcatToTextBlock check - #866

Merged
mernst merged 2 commits into
codespecs:masterfrom
mernst:errorprone-text-block
Oct 6, 2026
Merged

mernst merged 2 commits into
codespecs:masterfrom
mernst:errorprone-text-block

Conversation

@mernst

@mernst mernst commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Error Prone's StringConcatToTextBlock check suggests text blocks, which require Java 15 source code. Like PatternMatchingInstanceof (which requires Java 17 source code), disable it until Daikon's source code can use that feature.

It fires on the new test in #867.

@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

This review includes 1 billable file and costs up to $0.25.

Or wait 36 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: codespecs/daikon/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 2cfeeb0d-f7af-4555-b8dd-2cad65c74fd2
📥 Commits

Reviewing files that changed from the base of the PR and between b7136ac and e2d595b.

📒 Files selected for processing (1)
  • java/Makefile
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@mernst
mernst merged commit 779e30d into codespecs:master Oct 6, 2026
16 checks passed
@mernst
mernst deleted the errorprone-text-block branch October 6, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant