Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error prone warnings in :core-execution #28819

Merged
merged 29 commits into from May 3, 2024

Conversation

lptr
Copy link
Member

@lptr lptr commented Apr 15, 2024

Reviewing cheatsheet

Before merging the PR, comments starting with

  • ❌ ❓must be fixed
  • 🤔 💅 should be fixed
  • 💭 may be fixed
  • 🎉 celebrate happy things

@lptr lptr added a:chore Minor issue without significant impact in:execution-engine incremental, up-to-date, overlapping outputs labels Apr 15, 2024
@lptr lptr added this to the 8.9 RC1 milestone Apr 15, 2024
@lptr lptr self-assigned this Apr 15, 2024
@lptr lptr requested review from a team as code owners April 15, 2024 08:58
@lptr lptr requested review from cobexer and removed request for a team April 15, 2024 08:58
Copy link
Member

@asodja asodja left a comment

Choose a reason for hiding this comment

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

LGTM, added a small comment about annotations dependencies

@lptr lptr added this pull request to the merge queue Apr 15, 2024
@cobexer cobexer removed request for a team and cobexer April 15, 2024 10:10
@lptr lptr removed this pull request from the merge queue due to a manual request Apr 15, 2024
@lptr lptr added this pull request to the merge queue Apr 15, 2024
@blindpirate blindpirate removed this pull request from the merge queue due to a manual request Apr 16, 2024
@@ -72,11 +64,6 @@ public ImmutableList<String> getChangeDescriptions() {
public InputChangesInternal createInputChanges() {
return new NonIncrementalInputChanges(beforeExecutionState.getInputFileProperties(), incrementalInputProperties);
}

@Override
public BeforeExecutionState getBeforeExecutionState() {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 I love how code evaporates

@lptr lptr added this pull request to the merge queue Apr 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 22, 2024
@lptr lptr added this pull request to the merge queue Apr 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 23, 2024
@blindpirate blindpirate added this pull request to the merge queue Apr 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 24, 2024
@lptr lptr force-pushed the lptr/execution/fix-error-prone-warnings branch from b4d6a7c to 9ffa942 Compare April 26, 2024 16:02
@lptr lptr added this pull request to the merge queue Apr 26, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Apr 26, 2024
@lptr lptr requested a review from a team as a code owner May 1, 2024 12:21
@lptr lptr requested review from bamboo and abstratt and removed request for a team May 1, 2024 12:21
@lptr lptr added this pull request to the merge queue May 1, 2024
@bot-gradle
Copy link
Collaborator

The merge queue build has failed. Click here to see all failures.

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 1, 2024
@lptr lptr added this pull request to the merge queue May 3, 2024
@bot-gradle
Copy link
Collaborator

The merge queue build has failed. Click here to see all failures.

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 3, 2024
@cobexer
Copy link
Member

cobexer commented May 3, 2024

@bot-gradle merge

@gradle gradle deleted a comment from lptr May 3, 2024
@gradle gradle deleted a comment from lptr May 3, 2024
@bot-gradle bot-gradle added this pull request to the merge queue May 3, 2024
Merged via the queue into master with commit e75ecd1 May 3, 2024
28 checks passed
@bot-gradle bot-gradle deleted the lptr/execution/fix-error-prone-warnings branch May 3, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10 min review a:chore Minor issue without significant impact build-script-change in:execution-engine incremental, up-to-date, overlapping outputs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants