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

Clarified conflicting rules in multiple sections defined in the same .editorconfig file when matched for single file #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mipo256
Copy link
Member

@mipo256 mipo256 commented Mar 1, 2025

Solves #74


📚 Documentation preview 📚: https://editorconfig-specification--75.org.readthedocs.build/

@mipo256 mipo256 requested a review from xuhdev March 1, 2025 17:17
@mipo256 mipo256 added the clarification Improve text without changing its meaning label Mar 1, 2025
Comment on lines +215 to +218
files take precedence. Thus, in case a given file matches multiple sections
within a single ``.editorconfig`` file, the rules defined in the section that
comes later in the ``.editorconfig`` file take precedence over rules defined
in the section that comes earlier in the same ``.editorconfig`` file.
Copy link
Member

Choose a reason for hiding this comment

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

They look like should be put right after the first period of the paragraph. You can separate "If multiple EditorConfig ..." to a new paragraph.

Also, please change the word "rule" with "Pair". "Rule" is undefined but "Pair" is defined above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clarification Improve text without changing its meaning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants