Skip to content

Conversation

@leaysgur
Copy link
Member

@leaysgur leaysgur commented Aug 6, 2025

May fix #12844

At first glance, it looked like a simple change, so I created a PR.

But sorry, I think I'm not familiar with the context, so please feel free to close if not appropriate. 😓

@leaysgur leaysgur requested a review from Dunqing as a code owner August 6, 2025 09:04
@graphite-app
Copy link
Contributor

graphite-app bot commented Aug 6, 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-isolated-declarations Isolated Declarations C-enhancement Category - New feature or request labels Aug 6, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Aug 6, 2025

CodSpeed Instrumentation Performance Report

Merging #12845 will not alter performance

Comparing dts-module-to-ns (306b023) with main (5c7a258)

Summary

✅ 34 untouched benchmarks

@leaysgur
Copy link
Member Author

leaysgur commented Aug 7, 2025

@Dunqing Thanks for the review~! Updated. 👍🏻

@leaysgur leaysgur requested a review from Dunqing August 7, 2025 09:48
@Dunqing Dunqing added the 0-merge Merge with Graphite Merge Queue label Aug 7, 2025
Copy link
Member

Dunqing commented Aug 7, 2025

Merge activity

…namespace X` (#12845)

May fix #12844

At first glance, it looked like a simple change, so I created a PR.

But sorry, I think I'm not familiar with the context, so please feel free to close if not appropriate. 😓
@graphite-app graphite-app bot force-pushed the dts-module-to-ns branch from 8278190 to 306b023 Compare August 7, 2025 11:54
@graphite-app graphite-app bot merged commit 306b023 into main Aug 7, 2025
30 checks passed
@graphite-app graphite-app bot deleted the dts-module-to-ns branch August 7, 2025 12:00
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Aug 7, 2025
taearls pushed a commit to taearls/oxc that referenced this pull request Aug 12, 2025
…namespace X` (oxc-project#12845)

May fix oxc-project#12844

At first glance, it looked like a simple change, so I created a PR.

But sorry, I think I'm not familiar with the context, so please feel free to close if not appropriate. 😓
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-isolated-declarations Isolated Declarations C-enhancement Category - New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Oxc should rewrite module to namespace in isolated declarations emit

3 participants