Skip to content

release(oxlint): v0.15.7#8592

Merged
Boshen merged 1 commit intomainfrom
release/oxlint-1737247777
Jan 19, 2025
Merged

release(oxlint): v0.15.7#8592
Boshen merged 1 commit intomainfrom
release/oxlint-1737247777

Conversation

@oxc-bot
Copy link
Contributor

@oxc-bot oxc-bot commented Jan 19, 2025

[0.15.7] - 2025-01-19

Features

Bug Fixes

Performance

Refactor

Styling

@graphite-app
Copy link
Contributor

graphite-app bot commented Jan 19, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • 0-merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@github-actions github-actions bot added A-linter Area - Linter A-cli Area - CLI A-editor Area - Editor and Language Server labels Jan 19, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jan 19, 2025

Oxlint Ecosystem CI

suite result
rolldown/rolldown
napi-rs/napi-rs
toeverything/affine
preactjs/preact
microsoft/vscode
calcom/cal.com
bbc/simorgh
sidebase/nuxt-auth
elastic/kibana
DefinitelyTyped/DefinitelyTyped
Milkdown/milkdown
bluesky-social/social-app

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 19, 2025

CodSpeed Performance Report

Merging #8592 will not alter performance

Comparing release/oxlint-1737247777 (b9d67c5) with main (40f5165)

Summary

✅ 32 untouched benchmarks

@Boshen Boshen merged commit 66c8720 into main Jan 19, 2025
33 checks passed
@Boshen Boshen deleted the release/oxlint-1737247777 branch January 19, 2025 01:00
Copy link

@david1234121 david1234121 left a comment

Choose a reason for hiding this comment

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

DavidMashby.com

Copilot AI added a commit that referenced this pull request Jan 28, 2026
The minifier was incorrectly inlining expressions into for-in loops when
the loop variable was declared with `let` or `const`. This caused variable
shadowing issues where the inlined expression might reference a variable
with the same name as the for-in variable.

This fix adds a check to only allow inlining when the for-in variable is
declared with `var`, which is safe because `var` doesn't create a new scope.

Fixes #8592

Co-authored-by: sapphi-red <49056869+sapphi-red@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cli Area - CLI A-editor Area - Editor and Language Server A-linter Area - Linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants