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

lint configs should be shared across BE + FE #2

Open
jtrrll opened this issue Aug 22, 2024 · 1 comment
Open

lint configs should be shared across BE + FE #2

jtrrll opened this issue Aug 22, 2024 · 1 comment

Comments

@jtrrll
Copy link
Member

jtrrll commented Aug 22, 2024

This change is not strictly necessary, but a unique strength of this codebase is that TypeScript is used on both the frontend and backend. Language consistency will go a long way in speeding up development, but it is essential that your linters maintain that consistency (TypeScript can look very different under different linting rules!).

This may or may not be difficult to do, but reusing things like your ESLint and tsconfigs (if possible) across both stacks could be valuable. Not sure of the best way to implement this, maybe they can be directly reused or just kept in sync somehow...

@alisryan
Copy link
Collaborator

I can't find a way to do it besides having node in the root directory instead of separately in the frontend and backend, so I think it's a bit impractical to do at this point. I can look more into it though.

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

No branches or pull requests

2 participants