Skip to content

Make things threadsafe - #3

Closed
gpolaert wants to merge 1 commit into
masterfrom
dev
Closed

Make things threadsafe#3
gpolaert wants to merge 1 commit into
masterfrom
dev

Conversation

@gpolaert

@gpolaert gpolaert commented May 3, 2017

Copy link
Copy Markdown
Contributor

@renaudboutet Not very confident with thread-safe. Could you check?

@gpolaert gpolaert closed this May 3, 2017
@tylerbenson tylerbenson added this to the Closed milestone Apr 10, 2020
bantonsson pushed a commit that referenced this pull request Dec 7, 2021
Use gradle tool chains to build Java 11 only tests
bm1549 added a commit that referenced this pull request May 15, 2026
robertomonteromiguel added a commit that referenced this pull request Aug 31, 2026
Fixes 4 issues flagged in PR review:
- Quote/reword the "#3" criterion in the llm-validation suite so it
  doesn't start a YAML comment and break the parser.
- Sync the upstream core SKILL.md fix that grants the conventions and
  cross-sdk reviewers narrowly scoped read-only access instead of
  silently degrading to NOT VERIFIED under the read-only restriction.
- Sync the upstream core SKILL.md fix that stops the change-set target
  resolution from silently falling back to origin/master and instead
  asks for confirmation.
- Exempt cached Optional.empty()/OptionalInt.empty()/OptionalLong.empty()/
  OptionalDouble.empty() singletons from the J15 allocation rule in the
  performance override, avoiding guaranteed false positives.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.

2 participants