-
Notifications
You must be signed in to change notification settings - Fork 33
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): add unicorn/prefer-dom-node-text-content
rule
#2869
chore(eslint): add unicorn/prefer-dom-node-text-content
rule
#2869
Conversation
🎊 PR Preview c07ade7 has been successfully built and deployed to https://process-analytics-bpmn-visualization-js-doc_preview-pr-2869.surge.sh 🕐 Build time: 0.013s 🤖 By surge-preview |
♻️ PR Preview faa5b4f has been successfully destroyed since this PR has been closed. 🤖 By surge-preview |
43c83e7
to
c07ade7
Compare
c07ade7
to
c0ece7a
Compare
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.
✔️ Tested with the surge preview environment. https://process-analytics-bpmn-visualization-js-demo_preview-pr-2869.surge.sh/dev/public/: the version correctly displays in the footer
✔️ tested locally http://localhost:10001/dev/public/bpmn-rendering.html loading an invalid BPMN diagram
To avoid to have to many changes by enabling
plugin:unicorn/recommended
, I choose to enable some rule one-by-one.https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-dom-node-text-content.md
Part of #2824
Covers #2742