Skip to content

Conversation

@dschuff
Copy link
Member

@dschuff dschuff commented Oct 25, 2024

It still needs a human to approve, but now there's one less button to press.

@dschuff dschuff requested a review from sbc100 October 25, 2024 00:51
@dschuff dschuff marked this pull request as ready for review October 25, 2024 00:51
Copy link
Collaborator

@sbc100 sbc100 left a comment

Choose a reason for hiding this comment

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

I guess this still requires a human reviewer right?

@dschuff
Copy link
Member Author

dschuff commented Oct 25, 2024

Yes.
(I will try to test this in my fork before I land it here)

@dschuff
Copy link
Member Author

dschuff commented Nov 18, 2024

For some reason the PR creation doesn't work in my fork, going to try it on this PR

emscripten-bot and others added 2 commits November 18, 2024 15:07
Automated changes by
[create-pull-request](https://github.com/peter-evans/create-pull-request)
GitHub action

Co-authored-by: dschuff <[email protected]>
@dschuff
Copy link
Member Author

dschuff commented Nov 18, 2024

lol oops, this causes it to actually merge the PR...

@dschuff
Copy link
Member Author

dschuff commented Nov 19, 2024

OK, I think what actually happened here is that the test PR went to this branch instead of main, and there are no approval requirements for landing on this branch, so it just merged. The behavior I jsut saw on the emsdk side looked right.

@dschuff dschuff merged commit 3186de8 into main Nov 19, 2024
5 of 13 checks passed
@dschuff dschuff deleted the dschuff-patch-1 branch November 19, 2024 01:01
dschuff added a commit that referenced this pull request Nov 19, 2024
dschuff added a commit that referenced this pull request Nov 19, 2024
Reverts #22788

Oops, I left my testing code in.
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.

4 participants