Skip to content

Simplify .editorconfig#1045

Merged
mhucka merged 2 commits intoquantumlib:mainfrom
mhucka:overhaul-editorconfig
Apr 13, 2026
Merged

Simplify .editorconfig#1045
mhucka merged 2 commits intoquantumlib:mainfrom
mhucka:overhaul-editorconfig

Conversation

@mhucka
Copy link
Copy Markdown
Collaborator

@mhucka mhucka commented Apr 10, 2026

This simplifies .editorconfig by setting a global indent_size and only listing file types that deviate from that default.

mhucka added 2 commits April 10, 2026 19:25
Set a global indent_size default, and only list file types that differ.
@mhucka mhucka marked this pull request as ready for review April 10, 2026 19:27
@github-actions github-actions bot added the size: S 10< lines changed <50 label Apr 10, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the .editorconfig file by centralizing common indentation and line length settings under the global section and removing redundant per-file overrides. It also introduces specific line length limits for git commit messages. Feedback was provided regarding the Makefile configuration, noting that increasing the indentation size from 4 to 8 deviates from the existing project style and should be reverted to maintain consistency.

@mhucka mhucka requested review from fdmalone and sergeisakov April 10, 2026 20:54
Copy link
Copy Markdown
Collaborator

@sergeisakov sergeisakov left a comment

Choose a reason for hiding this comment

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

LGTM

@mhucka mhucka added this pull request to the merge queue Apr 13, 2026
Merged via the queue into quantumlib:main with commit b194c7e Apr 13, 2026
29 checks passed
@mhucka mhucka deleted the overhaul-editorconfig branch April 13, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants