-
Notifications
You must be signed in to change notification settings - Fork 355
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(eslint-plugin)!: remove dependencies from ESLint plugin #10752
Conversation
Preview: https://patternfly-react-pr-10752.surge.sh A11y report: https://patternfly-react-pr-10752-a11y.surge.sh |
fac6286
to
aef2228
Compare
Closes patternfly#10740 BREAKING CHANGE: this removes the bundled dependencies and pre-defined rules from the ESLint 'reccomended' configuration. Signed-off-by: Jon Koops <[email protected]>
aef2228
to
6c36df4
Compare
"eslint-config-prettier": "^9.1.0", | ||
"eslint-plugin-markdown": "^5.1.0", | ||
"eslint-plugin-prettier": "^5.1.3", | ||
"eslint-plugin-prettier": "^5.2.1", | ||
"eslint-plugin-react": "^7.34.2", | ||
"eslint-plugin-react-hooks": "^4.6.2", |
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.
Had to move some dependencies to the root, as they are used by the linting configuration of the project.
|
||
### Building | ||
|
||
``` | ||
yarn build | ||
``` | ||
|
||
Note the build scripts for this are located in the root package.json under `yarn build`. | ||
|
||
### Publishing | ||
|
||
``` | ||
yarn publish | ||
``` |
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.
These instructions do not apply to this package, they were likely copied from a template before, so I removed them here.
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.
We may want to double check that we don't need to set up any additional lint plugins / rule overrides in our config after these get removed.
I think this should not be an issue as the recommended rules from the plugin are not actually used by the configuration. |
Your changes have been released in: Thanks for your contribution! 🎉 |
Closes #10740
BREAKING CHANGE: this removes the bundled dependencies and pre-defined rules from the ESLint 'reccomended' configuration.