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

Add some proper tooling to the codebase #246

Open
Nuclear-Squid opened this issue Oct 3, 2024 · 0 comments
Open

Add some proper tooling to the codebase #246

Nuclear-Squid opened this issue Oct 3, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@Nuclear-Squid
Copy link
Owner

Being knee deep in the refactor for the analyzer (see #243), I’m realizing that the code base is starting to get too complex for us to keep doing stuff by hand, a bit of CI would be good.

I’m thinking we should get the triple combo of LSP / linter + auto-fomatter + unit tests, along with some type checking. We discussed the last subject with @fabi1cazenave IRL, but JSDoc could be a good start, as Typscript would add a bit too much complexity, mainly for our sister projects (mainly Kalamine).

Unit tests can be hard to implement before (#243) is merged, but the other three could be added in the mean time. I was thinking we could use eslint + prettier for JS and markdown-lint for the content of the site. Prettier is technically already in use in the code base, but it’s not linked to any kind of CI, so it was used once and never again (and that was a long time ago, before we switched over to Hugo).

I’m assigning you, @fabi1cazenave for this issue, but it’s mainly because you know this ecosystem a lot better than I do. I’ll gladly add those in if you don’t have much time.

@Nuclear-Squid Nuclear-Squid added the enhancement New feature or request label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants