Skip to content

Pull #18528: Fix skipMavenParsing #18531#18528

Closed
Pankraz76 wants to merge 1 commit into
checkstyle:masterfrom
Pankraz76:fix-skipMavenParsing-fix-pr
Closed

Pull #18528: Fix skipMavenParsing #18531#18528
Pankraz76 wants to merge 1 commit into
checkstyle:masterfrom
Pankraz76:fix-skipMavenParsing-fix-pr

Conversation

@Pankraz76
Copy link
Copy Markdown

@Pankraz76 Pankraz76 commented Jan 6, 2026

Pull #18528: Fix skipMavenParsing #18531

@Pankraz76 Pankraz76 changed the title Pull #18439: Fix skipMavenParsing Pull #18528: Fix skipMavenParsing Jan 6, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from 7231e5f to e2dec0b Compare January 6, 2026 14:59
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from e2dec0b to fee540d Compare January 6, 2026 15:00
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from fee540d to 9164c5a Compare January 6, 2026 15:02
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from 00a350a to 4e71f09 Compare January 6, 2026 15:03
@Pankraz76 Pankraz76 marked this pull request as ready for review January 6, 2026 15:03
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 6, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 9, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from 4e71f09 to fdff122 Compare January 9, 2026 12:15
@Pankraz76 Pankraz76 changed the title Pull #18528: Fix skipMavenParsing Pull #18528: Fix skipMavenParsing #18531 Jan 9, 2026
Comment thread pom.xml
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 9, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from fdff122 to dc3f122 Compare January 9, 2026 12:17
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 9, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from dc3f122 to 8351009 Compare January 9, 2026 12:17
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 9, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from 8351009 to eab0b12 Compare January 9, 2026 12:18
@Pankraz76
Copy link
Copy Markdown
Author

Too long with no output (exceeded 10m0s): context deadline exceeded

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 9, 2026
@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from eab0b12 to 97d0334 Compare January 9, 2026 15:25
@romani
Copy link
Copy Markdown
Member

romani commented Jan 9, 2026

Too much changes

@Pankraz76
Copy link
Copy Markdown
Author

yes just integrated to see if its working now. Small scope after rebase.

@Pankraz76 Pankraz76 force-pushed the fix-skipMavenParsing-fix-pr branch from 97d0334 to 281342b Compare January 10, 2026 10:05
Comment thread config/rewrite.yml
- org.openrewrite.java.migrate.UpgradeToJava17
- org.openrewrite.java.recipes.JavaRecipeBestPractices
- org.openrewrite.java.recipes.RecipeTestingBestPractices
- org.openrewrite.maven.BestPractices
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.

causing scm issue, we dont need it anyways.

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 10, 2026
@romani
Copy link
Copy Markdown
Member

romani commented Jan 11, 2026

No idea what is a problem from the beginning

@Pankraz76
Copy link
Copy Markdown
Author

lets fix if everything is breaking. we learn best though pain if not able to learn with brain.

@Pankraz76 Pankraz76 closed this Jan 11, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
romani pushed a commit that referenced this pull request Jan 11, 2026
@github-actions github-actions Bot added this to the 13.1.0 milestone Jan 11, 2026
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