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

LibWeb/Streams: Align a few AOs with current specification steps #3429

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

kennethmyhra
Copy link
Contributor

This aligns the following AOs with current spec steps:

  • ReadableByteStreamControllerFillPullIntoDescriptorFromQueue
  • ReadableByteStreamControllerRespondInClosedState
  • ReadableByteStreamControllerEnqueue

These are editoral changes from the PR: whatwg/streams#1336

@ladybird-bot
Copy link
Collaborator

Hello!

One or more of the commit messages in this PR do not match the Ladybird code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.

Align ReadableByteStreamControllerFillPullIntoDescriptorFromQueue with
current spec steps.
Align ReadableByteStreamControllerRespondInClosedState with current spec
steps.
Align ReadableByteStreamControllerEnqueue with current spec steps.
@kennethmyhra kennethmyhra force-pushed the editoral-after-commit-pull-into-descriptors-1326 branch from ce9bfe2 to c8bc1a6 Compare February 2, 2025 17:46
@kennethmyhra
Copy link
Contributor Author

This LibWeb: Align ReadableByteStreamControllerFillPullIntoDescriptorFromQueue became a character too long for the commit linter so I changed it to: LibWeb: Align ReadableByteStream(...)FillPullIntoDescriptorFromQueue. Not sure what we think of it 😆

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.

2 participants