-
Notifications
You must be signed in to change notification settings - Fork 12k
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
Migration to ESLint #13732
Comments
Hi, agreed however this is blocked, since at the moment ESLint is not yet feature complete. |
True. Should I link the necessary issues of TSLint and ESLint as well? |
That wouldn't hurt, though when |
I’m coordinating this with @alexeagle, stay tuned |
TSLint has been deprecated palantir/tslint#3865 (comment). |
@JamesHenry will the ESLint configuration be aligned with the Google JavaScript Style Guide, or any other style guide? |
Hi, Do you have some ideia if next versions will use ESlint as default? I'm still following the https://github.com/angular-eslint/angular-eslint. Thanks |
@JamesHenry Any progress on this? We just hit a milestone for TSlint deprecation
|
What's the planned timeline for this migration? |
Don't worry.I think Angular Team will fix it.As i say, Angular Team is a strong backing. |
@yy7054wyq5 I'm sure they will, but the question is when. @alan-agius4 @JamesHenry Any updates? |
Tslint to eslint migration is being targeted for version 10. |
A lot of us that work with Angular on Linux has tslint/vscode problems: |
How do you know, @alan-agius4? |
@LayZeeDK, because I am a member of the Angular Tooling Team. |
I don't understand why lint, unit test, and, e2e cannot be given as an option because these should be at the sole discretion of the developer. TSLint has been deprecated for more than a year. Protractor is not used in every Angular project, we can say the same about Jasmin. After all, endless arrangements in configuration files. I don't even want to talk about upgrading. So why do we have to do so? For 40MBs? |
@offwork I agree and even if it were not bundled, an option to configure that in an existing project should be given. |
Are you asking for an option to select linting and testing toolchain when running the |
@LayZeeDK an option to do it while running |
@offwork you can replace everything you want in an angular project NX is a great example that it's possible. The goal for angular was and is to provide a opinionated complete approach to app development. I am not saying that the problems you address are not real, but I still would prefer an opinionated approach that works out of the box instead of hours of figuring out configurations and how to update them (again the issues with Protractor are real, with tslint to eslint people work on it) |
Yes, I understand and, I use Nx too. However, it is not possible to use Nx in every organization for contractual employees like me. Angular CLI should be considered lightweight. And yes, as you said, all these should be given as options on the command line. Why? Here are my reasons:
|
@alan-agius4 how's the support for eslint looking in 11.0 release? |
Luckily, the Angular CLI is flexible enough to use almost any toolchain through builders. Believe me, I'm also tired of waiting like you. I'll leave TSLint behind in a smaller project to figure out where we're at. It's 5 seconds to midnight for TSLint. |
The tslint to eslint migration is currently being driven and worked on by the community and more information can be found here: https://github.com/angular-eslint/angular-eslint We are in touch with the maintainers of @angular-eslint to offer help and support to reduce the configuration boilerplate and complexity that’s currently required to switch over. That being said, it’s most likely that in future the CLI will not providing a linting solution out of the box, and it will be up to the users to choose a builder similar to how it's currently done for We’ll provide more details when we have them. |
Why is |
Because it’s no longer supported? How is that not a good reason on its own.
…On Fri, 30 Oct 2020 at 21:16, kerem ***@***.***> wrote:
Are you asking for an option to select linting and testing toolchain when
running the ng new command? You could use Nx if this is what you want.
Also, Angular version 11 is planned to have opt-in support (meaning an
experimental preview) for ESLint.
Yes, I understand and, I use Nx too. However, it is not possible to use Nx
in every organization for contractual employees like me. Angular CLI should
be considered lightweight. And yes, as you said, all these should be given
as options on the command line. Why? Here are my reasons:
1. The general population of the developer team has 2+ experience.
2. Novice developers are not familiar with Node Js command lines or
npm scripts.
3. For those who later get joined in the project like me, the
decisions in the frontend application are made by the backend dinosaurs.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#13732 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE36T4F4HOEHSHGS3EXG3J3SNK4BJANCNFSM4GY3QGPA>
.
|
TsLint is not but ESLint is. And I also cannot imagine not having some kind of linter as default in Angular. Just imagine all those new developers or dinosaur developer coming in Angular and using Angular without and TS/ESlint. Without default option build in they will not bother to install any alternatives. @LayZeeDK idea about making angular-eslint default linter is great I think. As he mention Angular was already using Codelyzer which is 3rd party tool anyway. |
Just FYI a schematic for converting your TSLint setup to ESLint will be available in beta in @angular-eslint at the end of next week |
Will It be working using existing
https://github.com/typescript-eslint/tslint-to-eslint-config or somerhing
else?
W dniu sob., 31.10.2020 o 10:15 James Henry <[email protected]>
napisał(a):
… Just FYI a schematic for converting your TSLint setup to ESLint will be
available in beta in @angular-eslint <https://github.com/angular-eslint>
at the end of next week
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#13732 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFKBQQLZP2KEEJEEBS4PESTSNPITBANCNFSM4GY3QGPA>
.
|
Yes it is a wrapper around that for the linter config translation logic, I have been pairing with Josh on making it work within a schematic context. It also contains the additional logic required around builder configuration and applying the converted rules appropriately via overrides, which the library doesn’t handle |
That's great news! I will make sure to use it at work when it will come out
of beta at least. So waiting till version 1.0! Thx for Your hard work )
sob., 31 paź 2020 o 10:31 James Henry <[email protected]> napisał(a):
… Yes it is a wrapper around that, I have been pairing with Josh on making
it work within a schematic context
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#13732 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFKBQQNU7K6P3X7ETA6TFYLSNPKPXANCNFSM4GY3QGPA>
.
|
Please see the README https://github.com/angular-eslint/angular-eslint for the long awaited |
Latest version is Angular 11 but still using tslint. Is eslint targeted for Angular 12 as a default? Or did I miss something? |
Angular will remove TSLint in some future version. It's been deprecated in Angular CLI version 11. TSLint now doesn't accept any issues or pull requests. Angular CLI will provide no out-of-the-box option to opt in to ESLint through a built-in migration or when setting up a new Angular workspace. You will have to migrate to or add ESLint and angular-eslint by following @JamesHenry's guide. |
angular-eslint is now v1.0.0 🚀 You can now create new Angular CLI workspaces without TSLint and Codelyzer if you so choose: https://jameshenry.blog/how-to-use-angular-eslint-in-a-new-project/ https://twitter.com/mrjameshenry/status/1344996625899532294?s=21 |
@JamesHenry kudos for the amazing job! Waiting for the blog post on how to migrate an existing project 😊 |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
🚀 Feature request
Migrate new projects to ESLint. This is a long-term goal.
Command (mark with an
x
)Description
Palantir decided to drop the further development of TSLint (read here).
So a long-term goal for the CLI should be to also migrate the newly created Angular Projects to ESLint. I opened an issue in the codelyzer project to think about migration as codelyzer is coupled to the CLI builds as well
Describe the solution you'd like
Newly created projects should use ESlint
The text was updated successfully, but these errors were encountered: