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.

Sign up for GitHub

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

Jump to bottom

Add anticipatedConcurrentIncoming[Bi/Uni]directionalStreams attributes & constructor arg #574

Merged

jan-ivar merged 15 commits into w3c:main from jan-ivar:maxstreams

Feb 27, 2024

Member

jan-ivar commented Nov 23, 2023 •

edited by pr-preview bot

Loading

Fixes #544 (for discussion).


          Add concurrentIncoming[Bi/Uni]directionalStreams constructor arg.

faa3b4c

jan-ivar requested review from vasilvv, jesup, wilaw and nidhijaju

November 23, 2023 00:34

jan-ivar self-assigned this

jan-ivar marked this pull request as draft

November 23, 2023 00:35

Member Author

jan-ivar commented Dec 6, 2023

Meeting:

add a setter


          Add a settable attribute and rename it to avoid confusion

beadf01

jan-ivar changed the title ~~Add concurrentIncoming[Bi/Uni]directionalStreams constructor arg.~~ Add anticipatedConcurrentIncoming[Bi/Uni]directionalStreams constructor arg.

jan-ivar changed the title ~~Add anticipatedConcurrentIncoming[Bi/Uni]directionalStreams constructor arg.~~ Add anticipatedConcurrentIncoming[Bi/Uni]directionalStreams attributes & constructor arg

jan-ivar marked this pull request as ready for review

January 17, 2024 14:55

jan-ivar added 2 commits

January 17, 2024 09:56


          Merge branch 'main' into maxstreams

b673ea4


          remove "incoming" on bidirectional to fix link

f230008

wilaw reviewed

View reviewed changes

index.bs Outdated Show resolved Hide resolved


          Consistent naming of internal slots

327d48c

Member Author

jan-ivar commented Jan 31, 2024

Meeting:

add ref to HTTP/2
long but plain name (bikeshed welcome!)
remove reference to MAX_FRAMES
present for next meeting for merging

jan-ivar commented

View reviewed changes

index.bs Outdated Show resolved Hide resolved

index.bs Outdated Show resolved Hide resolved

jan-ivar added 2 commits

January 31, 2024 10:15


          Don't mention QUIC

52cc556


          Don't mention QUIC

f3ee0fa

jan-ivar commented

View reviewed changes

index.bs Outdated Show resolved Hide resolved

index.bs Outdated Show resolved Hide resolved

jan-ivar added 2 commits

January 31, 2024 10:20


          Don't mention QUIC

6e17ee0


          Don't mention QUIC

853969c

jan-ivar commented

View reviewed changes

index.bs Outdated Show resolved Hide resolved


          Don't mention QUIC

1d9d5e4

jan-ivar commented

View reviewed changes

index.bs Outdated Show resolved Hide resolved


          Don't mention QUIC

ca9dc09

jan-ivar commented

View reviewed changes

index.bs Show resolved Hide resolved


          Add note to devs

4497c36

jan-ivar commented

View reviewed changes

index.bs Outdated Show resolved Hide resolved


          Fix error

bdb78bb

jan-ivar commented

View reviewed changes

index.bs Outdated Show resolved Hide resolved


          Remove indent

757df0b

jan-ivar added the Editors can integrate label


          Fix link ambiguity

701e688

jan-ivar merged commit 8db4c49 into w3c:main

2 checks passed

jan-ivar deleted the maxstreams branch

February 27, 2024 22:52

github-actions bot added a commit that referenced this pull request


          Merge pull request #574 from jan-ivar/maxstreams

ff31355

SHA: 8db4c49
Reason: push, by jan-ivar

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Editors can integrate