-
Notifications
You must be signed in to change notification settings - Fork 11
Closed
Labels
good first issueGood for newcomersGood for newcomers
Description
The following feedback was given for configuring a new Angular application with linting.
I wasn't sure which linters to configure. Are we meant to configure linters for JavaScript, Typescript, and Angular, or is just Angular sufficient? I think I eventually figured out that I need to configure Angular and JavaScript (as the Angular config talks about removing the TypeScript config, but I got hung up on this for a while). I think we could clarify this step.
Fix
- Clarify in the docs that the TS and Angular configurations include the TypeScript and JavaScript configs.
- Add an override for *.js to the docs for the Angular configuration
- Update the docs to mention the need to add something like this to
angular.json(after verifying that it's necessary for our current version of Angular)
"lintFilePatterns": [
"src/**/*.ts",
"src/**/*.html",
"acceptance-tests/**/*.ts",
"acceptance-tests/**/*.js"
]
rajsite and jattasNI
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers