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

Proposal to Replace Deprecated TSLint with ESLint, Prettier #193

Merged
merged 4 commits into from
Mar 24, 2024

Conversation

pplancq
Copy link
Contributor

@pplancq pplancq commented Mar 23, 2024

Hello,

I am proposing a significant change to our codebase. As TSLint is now deprecated, I suggest we replace it with ESLint and Prettier. I have used my personal configurations which are quite strict. However, I am open to discussion if there are any concerns or suggestions.

In addition, I have added Husky to our project. Husky will automatically run eslint --fix on every new file during the pre-commit hook. This will ensure that our code remains clean and adheres to our coding standards.

Please review these changes and provide your feedback.

Best.

@bencompton
Copy link
Owner

Thanks for contributing this! Getting the tooling up to date would be wonderful, and I should have some time to review this evening.

@bencompton
Copy link
Owner

I'm good with the tools you've introduced here and am also up for trying out your OSS dev-tools project. If you wouldn't mind getting the conflicts resolved (from merging your other PR), we can get this one merged as well. Thanks again!

@pplancq
Copy link
Contributor Author

pplancq commented Mar 24, 2024

Hello,

I'm rebasing my branch to correct the conflicts.
I'm glad to see you appreciate these changes.
Yes you could use my dev-tools project for your react projects that's why I made it public.

@bencompton bencompton merged commit 31d7160 into bencompton:master Mar 24, 2024
2 checks passed
@pplancq pplancq deleted the feature/eslint branch March 24, 2024 14:36
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.

None yet

2 participants