Skip to content

Apply unused io.spring.javaformat plugin #35094

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

Conversation

Pankraz76
Copy link

execute: gradle format

build is not fixing itself, impose burden on contributor, despite having auto fix solution in place but not activated, leaking quality and compliance.


* What went wrong:
Execution failed for task ':buildSrc:checkFormatMain'.
> Formatting violations found in the following files:
   * src/main/java/org/springframework/build/hint/RuntimeHintsAgentArgumentProvider.java
   * src/main/java/org/springframework/build/hint/RuntimeHintsAgentExtension.java
   * src/main/java/org/springframework/build/hint/RuntimeHintsAgentPlugin.java
   * src/main/java/org/springframework/build/KotlinConventions.java
   * src/main/java/org/springframework/build/ConventionsPlugin.java
   * src/main/java/org/springframework/build/shadow/ShadowSource.java
   * src/main/java/org/springframework/build/TestConventions.java
   * src/main/java/org/springframework/build/optional/OptionalDependenciesPlugin.java
   * src/main/java/org/springframework/build/architecture/ArchitectureCheck.java
   * src/main/java/org/springframework/build/architecture/ArchitecturePlugin.java
   * src/main/java/org/springframework/build/architecture/ArchitectureRules.java
   * src/main/java/org/springframework/build/SpringFrameworkExtension.java
   * src/main/java/org/springframework/build/CheckstyleConventions.java
   * src/main/java/org/springframework/build/JavaConventions.java
   * src/main/java/org/springframework/build/dev/LocalDevelopmentPlugin.java
   * src/main/java/org/springframework/build/multirelease/MultiReleaseJarPlugin.java
   * src/main/java/org/springframework/build/multirelease/MultiReleaseExtension.java
  
  Run `format` to fix.

@Pankraz76 Pankraz76 changed the title Apply unused io.spring.javaformat plugin Apply unused io.spring.javaformat plugin Jun 22, 2025
@Pankraz76 Pankraz76 force-pushed the pr-apply-lost-io.spring.javaformat branch from d335324 to 6828305 Compare June 22, 2025 12:27
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Jun 22, 2025
@@ -1,4 +1,4 @@
org.gradle.caching=true
assertjVersion=3.27.3
javaFormatVersion=0.0.43
Copy link
Author

Choose a reason for hiding this comment

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

why does renovate not take care about this?

@Pankraz76 Pankraz76 marked this pull request as ready for review June 22, 2025 12:30
@Pankraz76 Pankraz76 force-pushed the pr-apply-lost-io.spring.javaformat branch from 6828305 to abb3901 Compare June 22, 2025 12:42
}

repositories {
mavenCentral()
gradlePluginPortal()
}

tasks.named('compileJava') {
Copy link
Author

@Pankraz76 Pankraz76 Jun 22, 2025

Choose a reason for hiding this comment

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

how to bind this properly, please? @bclozel

only the checkFormat goal is executed by default, but not the format.

@bclozel bclozel closed this Jun 22, 2025
@bclozel bclozel added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jun 22, 2025
@bclozel
Copy link
Member

bclozel commented Jun 22, 2025

As stated in previous PRs, please stop submitting cosmetic changes that apply your own opinion.
This is something the project team should discuss and apply.

@Pankraz76
Copy link
Author

Understood.

Wondering why refer then on some plugin thats not used.
Assuming removal of dependency would work as well.

seems like technical depth making contributors fix whitespaces.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants