Skip to content

Issue #17487: Add StreamRulesRecipes#17782

Merged
romani merged 1 commit into
checkstyle:masterfrom
Pankraz76:StreamRulesRecipes-prerequisite-UpgradeToJava17
Sep 13, 2025
Merged

Issue #17487: Add StreamRulesRecipes#17782
romani merged 1 commit into
checkstyle:masterfrom
Pankraz76:StreamRulesRecipes-prerequisite-UpgradeToJava17

Conversation

@Pankraz76

Copy link
Copy Markdown

Issue #17487: Add StreamRulesRecipes

@Pankraz76 Pankraz76 marked this pull request as ready for review September 12, 2025 11:20
@Pankraz76 Pankraz76 force-pushed the StreamRulesRecipes-prerequisite-UpgradeToJava17 branch from 2bcdfd2 to dd5405c Compare September 12, 2025 11:21

@romani romani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot

@rdiachenko rdiachenko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@rdiachenko rdiachenko assigned romani and unassigned rdiachenko Sep 12, 2025
@romani

romani commented Sep 12, 2025

Copy link
Copy Markdown
Member

Please resolve conflict

@Pankraz76 Pankraz76 force-pushed the StreamRulesRecipes-prerequisite-UpgradeToJava17 branch from dd5405c to 0c92f6c Compare September 13, 2025 10:41
@Pankraz76

Copy link
Copy Markdown
Author

rebase, thanks for review.

@romani romani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks a lot

@romani romani merged commit 6c3053e into checkstyle:master Sep 13, 2025
120 checks passed
Comment thread pom.xml
<checkstyle.openrewrite.version>1.0.0-SNAPSHOT</checkstyle.openrewrite.version>
<rewrite-migrate-java.version>3.16.0</rewrite-migrate-java.version>
<rewrite-static-analysis.version>2.16.0</rewrite-static-analysis.version>
<rewrite-third-party.version>0.26.0</rewrite-third-party.version>

@Pankraz76 Pankraz76 Sep 13, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Pankraz76 , please remove in separate PR

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

resolved assuming by you thanks.

Pankraz76 pushed a commit to Pankraz76/maven that referenced this pull request Sep 22, 2025
Pankraz76 pushed a commit to Pankraz76/maven that referenced this pull request Sep 22, 2025
Pankraz76 pushed a commit to Pankraz76/maven that referenced this pull request Sep 22, 2025
Pankraz76 pushed a commit to Pankraz76/maven that referenced this pull request Sep 22, 2025
Pankraz76 pushed a commit to Pankraz76/maven that referenced this pull request Sep 23, 2025
Pankraz76 pushed a commit to Pankraz76/maven-parent that referenced this pull request Sep 23, 2025
gnodet pushed a commit to apache/maven that referenced this pull request Sep 25, 2025
- checkstyle/checkstyle#17782

Co-authored-by: Vincent Potucek <vpotucek@me.com>
gnodet pushed a commit to apache/maven that referenced this pull request Sep 26, 2025
- checkstyle/checkstyle#17782

Co-authored-by: Vincent Potucek <vpotucek@me.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.

3 participants