Skip to content

Set up SVLint - #35

Merged
TheDeepestSpace merged 22 commits into
mainfrom
svlint
Oct 11, 2025
Merged

Set up SVLint#35
TheDeepestSpace merged 22 commits into
mainfrom
svlint

Conversation

@TheDeepestSpace

Copy link
Copy Markdown
Member

See https://github.com/dalance/svlint/tree/master.

Haven't fixed any of the sources yet since there are quite a bit of offences. We'll just start fixing them as we go. We'll also allow this to fail in CI without breaking builds too.

@TheDeepestSpace
TheDeepestSpace force-pushed the svlint branch 3 times, most recently from 2c94365 to 5602c36 Compare July 16, 2025 02:46
@TheDeepestSpace TheDeepestSpace self-assigned this Jul 19, 2025
@TheDeepestSpace TheDeepestSpace linked an issue Jul 21, 2025 that may be closed by this pull request
@TheDeepestSpace

Copy link
Copy Markdown
Member Author

I'm not too happy with the amount of stylistic warnings that we get from this. I'll try to drop limit the styling to some bare minimum for now

@TheDeepestSpace

Copy link
Copy Markdown
Member Author

I think we might need to set this up sooner than later since we'll be getting a lot of new members soon and it would be nice to try to keep the code style consistent.

@TheDeepestSpace

Copy link
Copy Markdown
Member Author

I think I'll just not enable any lints yet so that its easier to do one by one.

@TheDeepestSpace
TheDeepestSpace merged commit 48358c4 into main Oct 11, 2025
4 checks passed
@TheDeepestSpace
TheDeepestSpace deleted the svlint branch May 7, 2026 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate SVlint

1 participant