Skip to content

tools: enable linter in test/fixtures/source-map/output - #57700

Merged
aduh95 merged 7 commits into
nodejs:mainfrom
aduh95:lint-source-map-output
Apr 4, 2025
Merged

tools: enable linter in test/fixtures/source-map/output#57700
aduh95 merged 7 commits into
nodejs:mainfrom
aduh95:lint-source-map-output

Conversation

@aduh95

Copy link
Copy Markdown
Contributor

No description provided.

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. labels Apr 1, 2025
@codecov

codecovBot commented Apr 1, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.24%. Comparing base (caf478b) to head (5944bf6).
Report is 3 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #57700 +/- ##
==========================================
- Coverage 90.25% 90.24% -0.01% 
==========================================
Files 630 630 Lines 184990 184990 Branches 36216 36213 -3 ==========================================
- Hits 166954 166950 -4 + Misses 11003 10993 -10 - Partials 7033 7047 +14 

see 19 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment threadeslint.config.mjs
@aduh95
aduh95 merged commit 87f8d3c into nodejs:mainApr 4, 2025
@aduh95

Copy link
Copy Markdown
ContributorAuthor

Landed in 87f8d3c

@aduh95
aduh95 deleted the lint-source-map-output branch April 4, 2025 13:56
Comment threadeslint.config.mjs
'!test/fixtures/console',
'!test/fixtures/errors',
'!test/fixtures/eval',
'!test/fixtures/source-map',

This comment was marked as off-topic.

RafaelGSS pushed a commit that referenced this pull request May 1, 2025
PR-URL: #57700
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
RafaelGSS pushed a commit that referenced this pull request May 2, 2025
PR-URL: #57700
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
aduh95 added a commit that referenced this pull request May 6, 2025
PR-URL: #57700
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
RafaelGSS pushed a commit that referenced this pull request May 14, 2025
PR-URL: #57700
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
aduh95 added a commit that referenced this pull request May 17, 2025
PR-URL: #57700
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
aduh95 added a commit that referenced this pull request May 19, 2025
PR-URL: #57700
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
@ghostghost mentioned this pull request Jun 8, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ciPRs that need a full CI run.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@aduh95@jasnell@MoLow@imronsman@nodejs-github-bot