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.

Sign up for GitHub

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

Jump to bottom

fix: ng lint default project when no project is supplied #12930

Closed
klauskpm opened this issue Nov 11, 2018 · 2 comments · Fixed by #12937
Closed

fix: ng lint default project when no project is supplied #12930

klauskpm opened this issue Nov 11, 2018 · 2 comments · Fixed by #12937
Labels
area: docs Related to the documentation freq1: low Only reported by a handful of users who observe it rarely severity1: confusing

Comments

@klauskpm
Copy link
Contributor

Bug Report or Feature Request (mark with an x)

- [x] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)

- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [x] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

Angular CLI: 7.0.1
Node: 10.10.0
OS: darwin x64
Angular: 7.0.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.10.1
@angular-devkit/build-angular      0.10.1
@angular-devkit/build-ng-packagr   0.10.1
@angular-devkit/build-optimizer    0.10.1
@angular-devkit/build-webpack      0.10.1
@angular-devkit/core               7.0.1
@angular-devkit/schematics         7.0.1
@angular/cli                       7.0.1
@ngtools/json-schema               1.1.0
@ngtools/webpack                   7.0.1
@schematics/angular                7.0.1
@schematics/update                 0.10.1
ng-packagr                         4.3.1
rxjs                               6.3.3
typescript                         3.1.3
webpack                            4.19.1

Repro steps

Create a new workspace, generate several projects and run ng lint. It will run in all projects and not just the default project, as specified in the docs: https://angular.io/cli/lint .

Desired functionality

Run only on the default project when a project name is not supplied.
@alan-agius4 alan-agius4 added the area: docs Related to the documentation label Nov 11, 2018
@alan-agius4
Copy link
Collaborator

alan-agius4 commented Nov 12, 2018 •
edited

Hi, thanks for reporting this. The docs here are incorrect, as when no project is provided it will run the lint for all projects within the workspace.

@alan-agius4 alan-agius4 added freq1: low Only reported by a handful of users who observe it rarely severity1: confusing labels Nov 12, 2018
@alan-agius4 alan-agius4 mentioned this issue Nov 12, 2018
Merged
kyliau pushed a commit that referenced this issue Nov 14, 2018
@alan-agius4
docs: update long descriptions for test, e2e and lint
26245c0 
Closes #12930
kyliau pushed a commit to kyliau/angular-cli that referenced this issue Nov 14, 2018
@alan-agius4 @kyliau
docs: update long descriptions for test, e2e and lint
7e9e3f1 
Closes angular#12930
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area: docs Related to the documentation freq1: low Only reported by a handful of users who observe it rarely severity1: confusing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: update long descriptions for test, e2e and lint
2 participants
@klauskpm @alan-agius4