Skip to content

chore(docs): refine JSDoc wording for experimental BindingMagicString#8639

Merged
graphite-app[bot] merged 1 commit intomainfrom
03-11-chore_add_experimental_js_doc_on_binding-magic-string_
Mar 11, 2026
Merged

chore(docs): refine JSDoc wording for experimental BindingMagicString#8639
graphite-app[bot] merged 1 commit intomainfrom
03-11-chore_add_experimental_js_doc_on_binding-magic-string_

Conversation

@IWANABETHATGUY
Copy link
Member

@IWANABETHATGUY IWANABETHATGUY commented Mar 11, 2026

Copy link
Member Author


How to use the Graphite Merge Queue

Add the label graphite: merge-when-ready to this PR to add it to the merge queue.

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.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@IWANABETHATGUY IWANABETHATGUY marked this pull request as ready for review March 11, 2026 11:26
Copilot AI review requested due to automatic review settings March 11, 2026 11:26
@netlify
Copy link

netlify bot commented Mar 11, 2026

Deploy Preview for rolldown-rs ready!

Name Link
🔨 Latest commit 7e1ef16
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/69b15bd99110e90008c9e50a
😎 Deploy Preview https://deploy-preview-8639--rolldown-rs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds TypeDoc/JSDoc documentation to the native Rust-backed MagicString binding so the public RolldownMagicString export is marked and described as experimental in generated API docs.

Changes:

  • Add a JSDoc block describing the native MagicString implementation.
  • Mark the binding-backed constructor export as @experimental.
  • Clarify that it is exported publicly as RolldownMagicString.

@IWANABETHATGUY IWANABETHATGUY changed the title chore: add experimental js doc on binding-magic-string chore(docs): refine JSDoc wording for experimental BindingMagicString Mar 11, 2026
@IWANABETHATGUY IWANABETHATGUY force-pushed the 03-11-chore_add_experimental_js_doc_on_binding-magic-string_ branch from 7fdef10 to 9b656b2 Compare March 11, 2026 11:45
Copy link
Contributor

Copilot AI commented Mar 11, 2026

@hyf0 I've opened a new pull request, #8641, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Member Author

IWANABETHATGUY commented Mar 11, 2026

Merge activity

  • Mar 11, 12:10 PM UTC: The merge label 'graphite: merge-when-ready' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Mar 11, 12:10 PM UTC: IWANABETHATGUY added this pull request to the Graphite merge queue.
  • Mar 11, 12:16 PM UTC: Merged by the Graphite merge queue.

@graphite-app graphite-app bot force-pushed the 03-11-chore_add_experimental_js_doc_on_binding-magic-string_ branch from 9b656b2 to 7e1ef16 Compare March 11, 2026 12:11
@graphite-app graphite-app bot merged commit 7e1ef16 into main Mar 11, 2026
30 checks passed
@graphite-app graphite-app bot deleted the 03-11-chore_add_experimental_js_doc_on_binding-magic-string_ branch March 11, 2026 12:16
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.

5 participants