-
-
Notifications
You must be signed in to change notification settings - Fork 960
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
Use 'TSLint' instead of 'ESLint'. #777
Conversation
Not sure where this file is used though. |
In fact, is was planned in the early stages with the extension. It was my idea. But I think it's obsolete. |
That's what I thought. @robertohuertasm Does this go to the |
This is from when we used |
Why did you merged this? Shouldn't it removed? 😄 |
Apparently, it has sentimental value to @robertohuertasm 🤣 |
Do you want to remove the file? I thought you were talking about the Eslint reference. |
nah, we want to remove the line. 😆 |
Why not remove the file entirely? |
Because maybe someone does not know the common conventions when creating PRs? |
PRs have their own |
So that where it's used, finally. So either we keep the line with reference to |
Hmm, I would let it there. So the PR is just fine. |
I'll keep the reference so they are advised to test it locally before arriving to CI. |
No description provided.