Skip to content

chore: suppress noisy Biome rules in Codacy#73

Merged
JohnnyVicious merged 1 commit into
mainfrom
chore/suppress-noisy-biome-rules
May 24, 2026
Merged

chore: suppress noisy Biome rules in Codacy#73
JohnnyVicious merged 1 commit into
mainfrom
chore/suppress-noisy-biome-rules

Conversation

@JohnnyVicious
Copy link
Copy Markdown
Owner

Summary

  • Adds biome.json disabling noNodejsModules and useTopLevelRegex
  • These two rules account for ~63% of Codacy findings (63 of 100 on page 1) but are false positives: this is a Node.js project and test regexes don't need top-level hoisting
  • Remaining 614 → ~230 actionable issues after suppression

Action required

After merge, enable "Use configuration file" for the Biome tool in Codacy's Code Patterns settings.

Test plan

  • Verify Codacy re-analyzes after merge and noisy rules no longer appear

Disable noNodejsModules and useTopLevelRegex — these generate ~63% of
Codacy findings but are false positives for a Node.js project with
inline test regexes.

Requires enabling "Use configuration file" for Biome in Codacy UI.
@sonarqubecloud
Copy link
Copy Markdown

@JohnnyVicious JohnnyVicious merged commit 7d5f0dd into main May 24, 2026
2 of 3 checks passed
@JohnnyVicious JohnnyVicious deleted the chore/suppress-noisy-biome-rules branch May 24, 2026 18:43
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

Copy link
Copy Markdown

@codacy-production codacy-production Bot left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR successfully introduces a biome.json configuration file to suppress the noNodejsModules and useTopLevelRegex rules, addressing identified false positives in the linting process. The implementation matches all defined acceptance criteria. Codacy results remain up to standards, and no merge-blocking issues or performance regressions were found.

Test suggestions

  • Verify that the biome.json file adheres to the official Biome schema and syntax.
  • Ensure the configuration correctly disables the specified rules when running biome lint locally.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Ensure the configuration correctly disables the specified rules when running `biome lint` locally.

TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback

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