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

Add no-useless-fallback-in-spread rule and update documentation #102

Merged
merged 12 commits into from
Dec 3, 2023

Conversation

AndreaPontrandolfo
Copy link
Owner

This pull request adds the no-useless-fallback-in-spread rule to the codebase. This rule helps to identify and remove unnecessary fallback values in spread syntax. Additionally, the documentation has been updated to include a new section on performance considerations. This section provides tips and strategies for optimizing linting performance, including benchmarking rules, disabling heavy rules, enabling rules only on CI, adopting ESLint cache, and revising glob patterns. This PR aims to improve code quality and provide developers with valuable information on optimizing linting performance.

Copy link

changeset-bot bot commented Nov 18, 2023

🦋 Changeset detected

Latest commit: ee22773

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
eslint-config-sheriff Patch
docs-website Patch
sheriff-webservices Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Nov 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sheriff ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 3, 2023 4:19pm

Copy link
Contributor

github-actions bot commented Nov 18, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 3 0 0.02s
⚠️ COPYPASTE jscpd yes 1 1.84s
⚠️ CSS stylelint 5 1 1.16s
✅ JSON eslint-plugin-jsonc 29 0 1.12s
⚠️ JSON jsonlint 29 1 0.25s
⚠️ JSON npm-package-json-lint yes 1 0.67s
✅ JSON prettier 29 0 0.97s
✅ JSON v8r 29 0 41.56s
⚠️ MARKDOWN markdownlint 35 41 1.34s
✅ MARKDOWN markdown-link-check 28 0 17.47s
✅ MARKDOWN markdown-table-formatter 28 0 0.28s
⚠️ REPOSITORY checkov yes 1 20.97s
⚠️ REPOSITORY gitleaks yes 1 0.2s
✅ REPOSITORY git_diff yes no 0.02s
✅ REPOSITORY grype yes no 10.8s
✅ REPOSITORY kics yes no 16.24s
✅ REPOSITORY secretlint yes no 1.05s
✅ REPOSITORY trivy yes no 5.31s
✅ REPOSITORY trivy-sbom yes no 1.33s
✅ REPOSITORY trufflehog yes no 3.83s
⚠️ SPELL cspell 90 84 11.22s
⚠️ YAML prettier 7 1 5.83s
✅ YAML v8r 7 0 7.64s
✅ YAML yamllint 7 0 3.41s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@AndreaPontrandolfo AndreaPontrandolfo merged commit 01a60aa into master Dec 3, 2023
4 checks passed
@AndreaPontrandolfo AndreaPontrandolfo deleted the perfs-docs branch December 3, 2023 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant