Skip to content

[CI] Configure EditorConfig for Java files#2740

Merged
jbampton merged 1 commit into
apache:masterfrom
jbampton:configure-java-for-editorconfig
Mar 12, 2026
Merged

[CI] Configure EditorConfig for Java files#2740
jbampton merged 1 commit into
apache:masterfrom
jbampton:configure-java-for-editorconfig

Conversation

@jbampton
Copy link
Copy Markdown
Member

We run our editorconfig checker with pre-commit and it found some Java files that had some tabs in the indentation.

Cleaned up the tabs converting to spaces.

Did you read the Contributor Guide?

Is this PR related to a ticket?

  • No:
    • this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

Enforcing standards with EditorConfig

How was this patch tested?

With pre-commit

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

We run our editorconfig checker with pre-commit and it found some Java files that had some tabs in the indentation.

Cleaned up the tabs converting to spaces.
@jbampton jbampton self-assigned this Mar 12, 2026
@jbampton jbampton added the java Pull requests that update Java code label Mar 12, 2026
Copy link
Copy Markdown
Member Author

@jbampton jbampton left a comment

Choose a reason for hiding this comment

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

Seems the Java code that had the mixed indentation was commented out.

@jbampton jbampton marked this pull request as ready for review March 12, 2026 12:07
@jbampton jbampton requested a review from jiayuasu as a code owner March 12, 2026 12:07
@jbampton jbampton merged commit d703c97 into apache:master Mar 12, 2026
41 checks passed
@jbampton jbampton deleted the configure-java-for-editorconfig branch March 12, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

java Pull requests that update Java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant