Skip to content

Comments

chore: make test-wasm32-unknown-unknown run unconditionally.#4624

Merged
graphite-app[bot] merged 1 commit intomainfrom
08-03-chore_make_test-wasm32-unknown-unknown_run_unconditionally
Aug 3, 2024
Merged

chore: make test-wasm32-unknown-unknown run unconditionally.#4624
graphite-app[bot] merged 1 commit intomainfrom
08-03-chore_make_test-wasm32-unknown-unknown_run_unconditionally

Conversation

@rzvxa
Copy link
Contributor

@rzvxa rzvxa commented Aug 3, 2024

It used to be unconditional before #4547, Having it to run on all PRs allows us to run oxc_ast::generated::assert_layouts tests on 32-bit platforms.

@graphite-app
Copy link
Contributor

graphite-app bot commented Aug 3, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

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

Copy link
Contributor Author

rzvxa commented Aug 3, 2024

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

Join @rzvxa and the rest of your teammates on Graphite Graphite

@overlookmotel overlookmotel added the 0-merge Merge with Graphite Merge Queue label Aug 3, 2024 — with Graphite App
@rzvxa rzvxa marked this pull request as ready for review August 3, 2024 15:24
@graphite-app
Copy link
Contributor

graphite-app bot commented Aug 3, 2024

Merge activity

It used to be unconditional before #4547, Having it to run all PRs allows us to run `oxc_ast::generated::assert_layouts` tests on 32-bit platforms.
@overlookmotel overlookmotel force-pushed the 08-03-chore_make_test-wasm32-unknown-unknown_run_unconditionally branch from d9e636d to 61e8b8c Compare August 3, 2024 15:25
@graphite-app graphite-app bot merged commit 61e8b8c into main Aug 3, 2024
@graphite-app graphite-app bot deleted the 08-03-chore_make_test-wasm32-unknown-unknown_run_unconditionally branch August 3, 2024 15:28
@overlookmotel
Copy link
Member

overlookmotel commented Aug 3, 2024

Contrary to the note above "overlookmotel added the merge label 5 hours ago — with Graphite App" I actually didn't add the merge label to this PR, or request merging it through Graphite app. I haven't used Graphite App for weeks.

Clearly Graphite made a mistake here. @Dunqing just to let you know maybe it was Graphite who merged your PR when no-one requested merging other week.

Luckily in this case, I do approve of the change and would have merged it.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants