Skip to content

Fix EC violations#164

Closed
szepeviktor wants to merge 4 commits intosanmai:mainfrom
szepeviktor:ec-fix
Closed

Fix EC violations#164
szepeviktor wants to merge 4 commits intosanmai:mainfrom
szepeviktor:ec-fix

Conversation

@szepeviktor
Copy link
Contributor

From #163

@sanmai
Copy link
Owner

sanmai commented May 13, 2025

Did you use a tool to implement these changes? I wonder if it can be used during CI.

@szepeviktor
Copy link
Contributor Author

@szepeviktor
Copy link
Contributor Author

Modern IDEs should be able to read EditorConfig.

@sanmai
Copy link
Owner

sanmai commented May 21, 2025

        -
            name: "Check EditorConfig configuration"
            run: "test -f .editorconfig"
        -
            name: "Check adherence to EditorConfig"
            uses: "greut/eclint-action@v0.5.0"

These are very cool! What do you have against adding these with your changes?

@szepeviktor
Copy link
Contributor Author

Nothing.

@sanmai
Copy link
Owner

sanmai commented Jan 6, 2026

Closing for no action.

@sanmai sanmai closed this Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants