Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

style(wasm): re-order imports #7864

Merged
merged 1 commit into from
Dec 14, 2024
Merged

Conversation

overlookmotel
Copy link
Contributor

@overlookmotel overlookmotel commented Dec 14, 2024

No description provided.

Copy link
Contributor Author

overlookmotel commented Dec 14, 2024


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.

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

@github-actions github-actions bot added the C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior label Dec 14, 2024
@overlookmotel overlookmotel marked this pull request as ready for review December 14, 2024 01:51
@overlookmotel
Copy link
Contributor Author

Trivial change, so merging without review.

@overlookmotel overlookmotel added the 0-merge Merge with Graphite Merge Queue label Dec 14, 2024
Copy link
Contributor Author

overlookmotel commented Dec 14, 2024

Merge activity

  • Dec 13, 8:52 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Dec 13, 8:52 PM EST: A user added this pull request to the Graphite merge queue.
  • Dec 13, 8:58 PM EST: The Graphite merge queue couldn't merge this PR because it had conflicts with the trunk branch.
  • Dec 13, 9:25 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Dec 13, 9:29 PM EST: A user added this pull request to the Graphite merge queue.
  • Dec 13, 9:34 PM EST: A user merged this pull request with the Graphite merge queue.

@overlookmotel overlookmotel changed the base branch from 12-14-refactor_semantic_syntax_wasm_remove_allow_non_snake_case_ to graphite-base/7864 December 14, 2024 01:52
Copy link

codspeed-hq bot commented Dec 14, 2024

CodSpeed Performance Report

Merging #7864 will not alter performance

Comparing 12-14-style_wasm_re-order_imports (3542b4f) with main (8cf9766)

Summary

✅ 29 untouched benchmarks

@graphite-app graphite-app bot changed the base branch from graphite-base/7864 to main December 14, 2024 01:57
@github-actions github-actions bot added the A-semantic Area - Semantic label Dec 14, 2024
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Dec 14, 2024
@overlookmotel overlookmotel force-pushed the 12-14-style_wasm_re-order_imports branch from 43dd295 to 353a73c Compare December 14, 2024 02:24
@overlookmotel overlookmotel added the 0-merge Merge with Graphite Merge Queue label Dec 14, 2024
@overlookmotel overlookmotel force-pushed the 12-14-style_wasm_re-order_imports branch from 353a73c to 3542b4f Compare December 14, 2024 02:30
@graphite-app graphite-app bot merged commit 3542b4f into main Dec 14, 2024
25 checks passed
@graphite-app graphite-app bot deleted the 12-14-style_wasm_re-order_imports branch December 14, 2024 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0-merge Merge with Graphite Merge Queue A-semantic Area - Semantic C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant