Skip to content
This repository was archived by the owner on Oct 24, 2023. It is now read-only.

Editorial: improve desc. of abstract operation #71

Merged
merged 2 commits into from
Nov 5, 2021

Conversation

jugglinmike
Copy link
Contributor

Following up on this conversation.

@syg
Copy link
Collaborator

syg commented Nov 1, 2021

Due to the update to using structured headers, this now needs a rebase. I'd be glad to rebase it myself but I don't have access to push to the upstream branch. @jugglinmike could you please rebase or give maintainers access to the PR?

@ljharb
Copy link
Member

ljharb commented Nov 1, 2021

Since it comes from bocoup's fork instead of mike's, they'd have to add you directly to the bocoup fork to provide that access.

@jugglinmike
Copy link
Contributor Author

@syg Updated!

@syg
Copy link
Collaborator

syg commented Nov 4, 2021

@syg Updated!

Thank you, but it looks like there're still conflicts that is blocking merge.

@jugglinmike
Copy link
Contributor Author

That's strange; GitHub is telling me that there are no conflicts:

image

@syg
Copy link
Collaborator

syg commented Nov 5, 2021

Oh I see, it was not possible to be rebased but it can be squashed.

@syg syg merged commit aa7114b into tc39:master Nov 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants