Skip to content
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

chore: add .md style in editorconfig #2525

Merged
merged 1 commit into from
Apr 16, 2024
Merged

chore: add .md style in editorconfig #2525

merged 1 commit into from
Apr 16, 2024

Conversation

returnToInnocence
Copy link
Contributor

Purpose of the PR

In view of the confusion caused by the truncation problem after markdown file formatting when you use the code format function of idea, this pr makes a correction. For markdown files, automatic line breaking is no longer carried out, and other formats are still in accordance with the default configuration of idea

Main Changes

  • add configuration max_line_length = off

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows:
    • After pulling the latest editorconfig file, close idea, open it again (this is very important), and enable code format to see if the markdown file is formatted as expected

Does this PR potentially affect the following parts?

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need
    • This is because the file format configuration is enabled by default

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 16, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 16, 2024
@VGalaxies VGalaxies requested a review from imbajin April 16, 2024 13:02
indent_style = space
indent_size = 4

[*.{java, xml}]
Copy link
Member

Choose a reason for hiding this comment

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

no idea why remove the space but could ignore it~

@imbajin imbajin merged commit 7d335b2 into apache:master Apr 16, 2024
14 checks passed
VGalaxies pushed a commit that referenced this pull request Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants