Skip to content

Conversation

@wadoon
Copy link
Member

@wadoon wadoon commented Jul 7, 2024

Remove the version lock for JDT in the spotless configurations, b/c the new version can handle all classes w/o error.

Also, apply spotless to the code basis. It seems, that Java in Javadoc is also formatted.

@wadoon wadoon requested a review from WolframPfeifer July 7, 2024 13:30
@wadoon wadoon self-assigned this Jul 7, 2024
@wadoon wadoon added the 🛠 Maintenance Code quality and related things w/o functional changes label Jul 7, 2024
@wadoon wadoon added this to the v2.14.0 milestone Jul 7, 2024
@wadoon wadoon enabled auto-merge August 4, 2024 11:32
@wadoon wadoon force-pushed the weigl/spotlessjdtversion branch from 78d7608 to 8c12755 Compare August 24, 2024 10:18
@wadoon wadoon force-pushed the weigl/spotlessjdtversion branch from 8c12755 to c150fca Compare October 25, 2024 12:18
@wadoon wadoon force-pushed the weigl/spotlessjdtversion branch from c150fca to e44c969 Compare November 17, 2024 21:59
Copy link
Member

@WolframPfeifer WolframPfeifer left a comment

Choose a reason for hiding this comment

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

I agree that since the eclipse formatter does not crash any more with the newer version, we can remove the version lock. The change with the indentation in a few places is good. However, I don't think we should reformat the code inside <pre> tags in JavaDoc. Is there are way to switch that off?

@wadoon wadoon force-pushed the weigl/spotlessjdtversion branch from e44c969 to e3b5cc1 Compare November 22, 2024 17:20
@wadoon wadoon disabled auto-merge November 22, 2024 17:20
@wadoon wadoon force-pushed the weigl/spotlessjdtversion branch from e3b5cc1 to 9a88c26 Compare November 27, 2024 17:21
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠 Maintenance Code quality and related things w/o functional changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants