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

Jump to bottom

Designate ngtcp2 and its dependency versions #10257

Closed
wants to merge 1 commit into from
Closed

Designate ngtcp2 and its dependency versions #10257

wants to merge 1 commit into from

Conversation

tatsuhiro-t
Copy link
Contributor

Designate ngtcp2 and its dependency versions so that the CI build does not fail without our control.

@tatsuhiro-t
Designate ngtcp2 and its dependency versions
628be06 
Designate ngtcp2 and its dependency versions so that the CI build does
not fail without our control.
bagder
bagder approved these changes Jan 8, 2023
View reviewed changes
Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

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

👍 will make us less sensitive for changes upstream

@bagder bagder added HTTP/3 h3 or quic related CI Continuous Integration labels Jan 8, 2023
@tatsuhiro-t tatsuhiro-t marked this pull request as ready for review January 8, 2023 12:54
mback2k
mback2k approved these changes Jan 8, 2023
View reviewed changes
Copy link
Member

@mback2k mback2k left a comment

Choose a reason for hiding this comment

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

Makes sense 👍

@bagder bagder closed this in 3022eea Jan 8, 2023
@bagder
Copy link
Member

bagder commented Jan 8, 2023

Thanks!

@bagder bagder deleted the ngtcp2-release-versions branch January 8, 2023 19:33
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
@tatsuhiro-t @bch
GHA: use designated ngtcp2 and its dependencies versions
1221beb 
Designate ngtcp2 and its dependency versions so that the CI build does
not fail without our control.

Closes curl#10257
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bagder bagder bagder approved these changes

@mback2k mback2k mback2k approved these changes

@cmeister2 cmeister2 Awaiting requested review from cmeister2

Assignees
No one assigned
Labels
CI Continuous Integration HTTP/3 h3 or quic related
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@tatsuhiro-t @bagder @mback2k