feat(nx): add ability to ignore files from affected commands

[wesleygrimes]

Check .nxignore for globs to add to ignored files in affected commands

Please make sure you have read the submission guidelines before posting an PR

Please make sure that your commit message follows our format.

Example: fix(nx): must begin with lowercase

Current Behavior (This is the behavior we have today, before the PR is merged)

Affected commands ignore files/folders in .gitignore

Expected Behavior (This is the new behavior we can expect after the PR is merged)

Affected commands ignore files/folders in .gitignore and .nxignore

Issue

Implement #895

[wesleygrimes]

This is WIP, need to update documentation to mention new ability to provide .nxignore and ignore property in nx.json.

[wesleygrimes]

@vsavkin updated react and web guides to mention ignore feature

[wesleygrimes]

Looks like the build/test on travis failing because of some cached files for cypress.

[brandonroberts]

You can delete the cache for your PR by clicking on the "More Options" menu and going to caches. Then restart the job

[wesleygrimes]

You can delete the cache for your PR by clicking on the "More Options" menu and going to caches. Then restart the job

Looks like I don't have access rights to do that. Unless I am missing something.

[vsavkin]

@wesleygrimes could you rebase it? Sorry about it.

[wesleygrimes]

@wesleygrimes could you rebase it? Sorry about it.

Sure. Will do.

[vsavkin]

I don't think we should support both .nxignore and the ignore property in nx.json. They do the same thing.

I'd just add support for .nxignore. Then, the PR will be a lot smaller.

[wesleygrimes]

Ready for review

[johannesschobel]

This, somehow does not work for me. I added a .nxignore file in the root-folder of my workspace and added the line **/generated in order to exclude all files in generated folders in my projects.
While this works for the .gitignore, the generated/*.ts files are still linted during my npm run affected:lint command - resulting in errors :(

Any solution for this?

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.