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.

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/consistent-function-scoping rule #2873

Merged
merged 7 commits into from
Sep 21, 2023

Conversation

csouchet
Copy link
Member

@csouchet csouchet commented Sep 20, 2023

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/consistent-function-scoping.md

Part of #2824 

Covers #2742

@csouchet csouchet added chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first refactoring Code refactoring labels Sep 20, 2023
@github-actions
Copy link

github-actions bot commented Sep 20, 2023

♻️ PR Preview 1f90bed has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

@github-actions
Copy link

github-actions bot commented Sep 20, 2023

🎊 PR Preview 1f90bed has been successfully built and deployed to https://process-analytics-bpmn-visualization-js-doc_preview-pr-2873.surge.sh

🕐 Build time: 0.017s

🤖 By surge-preview

…/consistent-function-scoping

(cherry picked from commit 356891b4eb9659d019ccfc1a696d89f4eedd6cc6)
…tent-function-scoping

(cherry picked from commit 24db8933d401c73da3bb4413a97f07af1837e5ab)
-  Move function 'newIconStyleConfiguration' to the outer scope  unicorn/consistent-function-scoping
-  Move function 'newShapeConfiguration' to the outer scope      unicorn/consistent-function-scoping
-  Move function 'expectSize' to the outer scope                 unicorn/consistent-function-scoping

(cherry picked from commit 900a6dbe7658f2f10d079886377e96bc814c3c94)
- Move function 'getShapeDirectory' to the outer scope  unicorn/consistent-function-scoping
- Move function 'getEdgeDirectory' to the outer scope   unicorn/consistent-function-scoping
- Move function 'getShapeDirectory' to the outer scope  unicorn/consistent-function-scoping

(cherry picked from commit 831c8eafcff57500b772e44f4b8c33e4c2105400)
… scope unicorn/consistent-function-scoping

(cherry picked from commit 62f3e25ff9c5444e0d1c592db4053c2b0f877944)
…nsistent-function-scoping

(cherry picked from commit 2d21603cb3f19d792e47148bfb0694da780ada40)
@csouchet csouchet force-pushed the 2742-add_unicorn/consistent-function-scoping_rule branch from 5a8b86e to 1f90bed Compare September 21, 2023 15:54
@csouchet csouchet removed the depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first label Sep 21, 2023
@sonarcloud
Copy link

sonarcloud bot commented Sep 21, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@csouchet csouchet marked this pull request as ready for review September 21, 2023 15:58
Copy link
Member

@tbouffard tbouffard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✔️ All checks pass

@csouchet csouchet merged commit 2ad3fe6 into master Sep 21, 2023
26 checks passed
@csouchet csouchet deleted the 2742-add_unicorn/consistent-function-scoping_rule branch September 21, 2023 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) refactoring Code refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants