Skip to content

chore(deps): update dependency globals to v17 - #1608

Merged
kriszyp merged 1 commit into
mainfrom
renovate/globals-17.x
Jul 6, 2026
Merged

chore(deps): update dependency globals to v17#1608
kriszyp merged 1 commit into
mainfrom
renovate/globals-17.x

Conversation

@renovate

@renovate renovate Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
globals ^16.5.0^17.0.0 age confidence

Release Notes

sindresorhus/globals (globals)

v17.7.0

Compare Source

v17.6.0

Compare Source


v17.5.0

Compare Source


v17.4.0

Compare Source


v17.3.0

Compare Source


v17.2.0

Compare Source

  • jasmine: Add throwUnless and throwUnlessAsync globals (#​335) 97f23a7

v17.1.0

Compare Source


v17.0.0

Compare Source

Breaking
Improvements


Configuration

📅 Schedule: (in timezone America/New_York)

  • Branch creation
    • "before 9am on Monday"
  • 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.

@socket-security

socket-security Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedglobals@​16.5.0 ⏵ 17.7.0100 +110086 +192100

View full report

@renovate
renovate Bot force-pushed the renovate/globals-17.x branch 2 times, most recently from c75fde7 to 9b43da9 Compare July 6, 2026 12:23

@dawsontoth dawsontoth left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed via automated Renovate check.

Diff: globals ^16.5.0^17.0.0 (resolved 17.7.0) — a dev-only lint-tooling dependency + lockfile. No source changes.

Local verification (Node 24, npm ci):

  • lint (oxlint): 7 problems (1 error, 6 warnings) — byte-for-byte identical to the main baseline. The globals v17 bump introduces zero new lint problems.
  • format:check (prettier): ✅ clean.

On the red CI: the failing runLinter and Unit Test checks are pre-existing on main — the latest push to main is itself red on both (Lint fails on integrationTests/resources/allowread-fail-closed.test.ts:14 no-restricted-imports for node:assert/strict; Unit Test fails on flaky event-ordering assertions "expected at least 1 event for id=1, got 0"). None of this is caused by this bump, and globals (a dev lint-config helper) cannot affect the runtime unit tests. Those failures need a separate fix on main; a rebase will carry it in once main is green.

The dependency change itself is safe — approving. (Merge will still gate on the branch going green after main's pre-existing lint/unit breakage is resolved.)

@renovate
renovate Bot force-pushed the renovate/globals-17.x branch from 9b43da9 to d4d1ba2 Compare July 6, 2026 14:18

@kriszyp kriszyp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed via Claude review-queue — devDependency-only globals v16->v17 bump, unused directly by the oxlint config. CI failure is the known pre-existing node:assert/strict lint issue (fix already merged in #1612), unrelated to this PR. LGTM.

@kriszyp
kriszyp merged commit 80af512 into main Jul 6, 2026
45 of 48 checks passed
@kriszyp
kriszyp deleted the renovate/globals-17.x branch July 6, 2026 14:56
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.

2 participants