Skip to content

chore(devdeps): update dependency @eslint/js to ~9.38.0#458

Merged
ryanbas21 merged 1 commit into
mainfrom
renovate/eslint-monorepo
Oct 28, 2025
Merged

chore(devdeps): update dependency @eslint/js to ~9.38.0#458
ryanbas21 merged 1 commit into
mainfrom
renovate/eslint-monorepo

Conversation

@renovate
Copy link
Copy Markdown
Contributor

@renovate renovate Bot commented Oct 24, 2025

This PR contains the following updates:

Package Change Age Confidence
@eslint/js (source) ~9.37.0 -> ~9.38.0 age confidence

Release Notes

eslint/eslint (@​eslint/js)

v9.38.0

Compare Source

Features
  • ce40f74 feat: update complexity rule to only highlight function header (#​20048) (Atul Nair)
  • e37e590 feat: correct no-loss-of-precision false positives with e notation (#​20187) (Francesco Trotta)
Bug Fixes
  • 50c3dfd fix: improve type support for isolated dependencies in pnpm (#​20201) (Francesco Trotta)
  • a1f06a3 fix: correct SourceCode typings (#​20114) (Pixel998)
Documentation
  • 462675a docs: improve web accessibility by hiding non-semantic character (#​20205) (루밀LuMir)
  • c070e65 docs: correct formatting in no-irregular-whitespace rule documentation (#​20203) (루밀LuMir)
  • b39e71a docs: Update README (GitHub Actions Bot)
  • cd39983 docs: move custom-formatters type descriptions to nodejs-api (#​20190) (Percy Ma)
Chores

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Oct 24, 2025

⚠️ No Changeset found

Latest commit: 4382cca

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 24, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Oct 24, 2025

View your CI Pipeline Execution ↗ for commit 4382cca

Command Status Duration Result
nx run-many -t build ✅ Succeeded <1s View ↗
nx affected -t build typecheck lint test e2e-ci ✅ Succeeded 1m 21s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-28 19:52:26 UTC

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 19.05%. Comparing base (a5fc6d7) to head (4382cca).

❌ Your project status has failed because the head coverage (19.05%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff            @@
##           main     #458       +/-   ##
=========================================
+ Coverage      0   19.05%   +19.05%     
=========================================
  Files         0      138      +138     
  Lines         0    27477    +27477     
  Branches      0      975      +975     
=========================================
+ Hits          0     5237     +5237     
- Misses        0    22240    +22240     

see 138 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Oct 24, 2025

Open in StackBlitz

@forgerock/davinci-client

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/davinci-client@458

@forgerock/oidc-client

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/oidc-client@458

@forgerock/protect

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/protect@458

@forgerock/sdk-types

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/sdk-types@458

@forgerock/sdk-utilities

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/sdk-utilities@458

@forgerock/iframe-manager

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/iframe-manager@458

@forgerock/sdk-logger

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/sdk-logger@458

@forgerock/sdk-oidc

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/sdk-oidc@458

@forgerock/sdk-request-middleware

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/sdk-request-middleware@458

@forgerock/storage

pnpm add https://pkg.pr.new/ForgeRock/ping-javascript-sdk/@forgerock/storage@458

commit: 4382cca

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 24, 2025

Deployed 9f44a0c to https://ForgeRock.github.io/ping-javascript-sdk/pr-458/9f44a0c81b77070cf9de77837998eafe30095720 branch gh-pages in ForgeRock/ping-javascript-sdk

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 24, 2025

📦 Bundle Size Analysis

📦 Bundle Size Analysis

🚨 Significant Changes

🔻 @forgerock/journey-client - 0.0 KB (-82.0 KB, -100.0%)

📊 Minor Changes

📈 @forgerock/journey-client - 82.0 KB (+0.0 KB)

➖ No Changes

@forgerock/device-client - 9.2 KB
@forgerock/oidc-client - 23.0 KB
@forgerock/protect - 150.1 KB
@forgerock/sdk-utilities - 7.5 KB
@forgerock/sdk-types - 8.0 KB
@forgerock/storage - 1.4 KB
@forgerock/sdk-logger - 1.6 KB
@forgerock/iframe-manager - 2.4 KB
@forgerock/sdk-request-middleware - 4.4 KB
@forgerock/sdk-oidc - 2.5 KB
@forgerock/davinci-client - 35.4 KB


13 packages analyzed • Baseline from latest main build

Legend

🆕 New package
🔺 Size increased
🔻 Size decreased
➖ No change

ℹ️ How bundle sizes are calculated
  • Current Size: Total gzipped size of all files in the package's dist directory
  • Baseline: Comparison against the latest build from the main branch
  • Files included: All build outputs except source maps and TypeScript build cache
  • Exclusions: .map, .tsbuildinfo, and .d.ts.map files

🔄 Updated automatically on each push to this PR

@renovate renovate Bot force-pushed the renovate/eslint-monorepo branch 2 times, most recently from e159050 to 4377a21 Compare October 27, 2025 17:19
Copy link
Copy Markdown
Contributor

@nx-cloud nx-cloud Bot left a comment

Choose a reason for hiding this comment

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

Nx Cloud has identified a possible root cause for your failed CI:

The failure was classified as not being a code change because the pull request only modifies a development dependency version in package.json (upgrading @eslint/js from ~9.37.0 to ~9.38.0). This is a tooling update that should not affect runtime application behavior or test outcomes.

The failing test is an end-to-end Playwright test that attempts to verify a user interface element becomes visible after clicking a "Sign On" button. The test expects to find text content "Complete" on the page but encounters a timeout because the element is not found within 5000ms.

This type of failure - where a UI element fails to appear during an E2E test - is characteristic of timing issues, race conditions, or environmental instability in test execution rather than a direct consequence of the ESLint version bump. The ESLint package is a static code analysis tool used during development and does not influence the runtime behavior of the application or the rendering of UI elements during test execution.

The change to @eslint/js is purely a linting tool upgrade in devDependencies. Such updates do not modify application logic, component rendering, API responses, or any other runtime characteristics that would cause a UI element to fail appearing in an E2E test scenario.

Therefore, this failure appears to be unrelated to the code changes in the PR and is more likely attributable to test flakiness, environmental conditions, or timing issues in the test infrastructure rather than the ESLint version update itself.

A code change would likely not resolve this issue, so no action was taken.

Nx CloudView in Nx Cloud ↗


🎓 To learn more about Self Healing CI, please visit nx.dev

@renovate renovate Bot force-pushed the renovate/eslint-monorepo branch from 4377a21 to 4382cca Compare October 28, 2025 19:49
@ryanbas21 ryanbas21 merged commit 06f2776 into main Oct 28, 2025
8 checks passed
@ryanbas21 ryanbas21 deleted the renovate/eslint-monorepo branch October 28, 2025 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants