Skip to content

Create AvoidEagerApis and surrounding infrastructure #275

Open
kelvinou01 wants to merge 1 commit into
developfrom
okelvin/ban-eager-methods
Open

Create AvoidEagerApis and surrounding infrastructure #275
kelvinou01 wants to merge 1 commit into
developfrom
okelvin/ban-eager-methods

Conversation

@kelvinou01
Copy link
Copy Markdown
Contributor

Before this PR

This PR adds

  • An error-prone to ban common ways eagerness "sprouts" from the codebase, to prevent more eagerness from appearing in all Palantir gradle code
  • Surrounding infrastructure for matching + fixing (similar to IllegalMethodCalledDuringTaskExecution)
  • Two example autofixes to demonstrate said infrastructure. The autofixes are gated behind BestEffortMode
  • 500 lines of tests

After this PR

==COMMIT_MSG==
Create AvoidEagerApis and surrounding infrastructure
==COMMIT_MSG==

Possible downsides?

@changelog-app
Copy link
Copy Markdown

changelog-app Bot commented Nov 10, 2025

Generate changelog in changelog/@unreleased

Type (Select exactly one)

  • Feature (Adding new functionality)
  • Improvement (Improving existing functionality)
  • Fix (Fixing an issue with existing functionality)
  • Break (Creating a new major version by breaking public APIs)
  • Deprecation (Removing functionality in a non-breaking way)
  • Migration (Automatically moving data/functionality to a new system)

Description

Create AvoidEagerApis and surrounding infrastructure

Check the box to generate changelog(s)

  • Generate changelog entry

@changelog-app
Copy link
Copy Markdown

changelog-app Bot commented Nov 10, 2025

Successfully generated changelog entry!

Need to regenerate?

Simply interact with the changelog bot comment again to regenerate these entries.


📋Changelog Preview

✨ Features

  • Create AvoidEagerApis and surrounding infrastructure (#275)

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