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
chore: update eslint and nx deps #3371
Conversation
✅ Deploy Preview for ngrx-io canceled.
|
@@ -113,7 +121,6 @@ | |||
"undefined" | |||
], | |||
"id-match": "error", | |||
"import/no-deprecated": "warn", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed because this was taking too long 😅
@@ -636,7 +636,7 @@ | |||
"outputs": ["coverage/modules/schematics"] | |||
}, | |||
"build-package": { | |||
"builder": "@nrwl/node:package", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed in nrwl/nx#9086
87d7f0c
to
2f0a490
Compare
@@ -235,7 +235,7 @@ jobs: | |||
- write_master_hash | |||
- run: | |||
name: Run Affected E2E Tests | |||
command: yarn nx affected --target=e2e --base=$(cat ~/project/master.txt) --head=$CIRCLE_SHA1 --headless --parallel | |||
command: yarn nx affected --target=e2e --base=$(cat ~/project/master.txt) --head=$CIRCLE_SHA1 --parallel --exclude=docs-app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why docs-app
is getting picked up as a project that has the e2e
target, that's why I just excluded it here 😅
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Closes #
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Before I started the migratation of the ESLint plugin, I wanted this repo to be on the latest ESLint version (8). By doing that, I also had to update the NX dependencies.
I also had to make some changes to make the
ng lint
happy.