-
Notifications
You must be signed in to change notification settings - Fork 4
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(eslint-config): remove formatting rules #14
Conversation
As discussed in #13 |
We are manually enabling |
4f39751
to
09ed232
Compare
@spike-rabbit Removed |
since |
09ed232
to
5ca6d24
Compare
@spike-rabbit Done. |
5ca6d24
to
5cfa901
Compare
5cfa901
to
a9842d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
3362147
to
39b9cd1
Compare
BREAKING CHANGE: `eslint-config-typescript` and `eslint-config-angular` no longer ship any formatting rules, please use prettier or set up formatting rules yourself.
39b9cd1
to
99fe152
Compare
This is merged on next, but it seems like github does not recognize this as I rebased locally. |
BREAKING CHANGE:
eslint-config-typescript
andeslint-config-angular
no longer ship any formatting rules, please use prettier or set up formatting rules yourselfRemoves the non-functional formatting rules