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

fix: remove custom eslint from TS template #888

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Conversation

kabaros
Copy link
Contributor

@kabaros kabaros commented Oct 25, 2024

now that eslint for TS is integrated in cli-style, no need to do anything special for linting eslint

@kabaros kabaros requested a review from a team October 25, 2024 15:35
now that eslint for TS is integrated in cli-style, no need to do anything special for linting eslint
@kabaros
Copy link
Contributor Author

kabaros commented Oct 25, 2024

consumers can install eslint like they do in the current JS template with d2 style add eslint - although I think it'd be a good idea to add these by default to both JS and TS templates?

@kabaros kabaros merged commit 71cef4b into alpha Oct 28, 2024
6 checks passed
@kabaros kabaros deleted the update/ts-template-eslint branch October 28, 2024 14:37
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 12.0.0-alpha.20 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants