Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

kainino0x
Copy link
Contributor

@kainino0x kainino0x commented Mar 13, 2020

@kvark just a few nits. please make sure it still matches your intent.


💥 Error: write EPROTO 140222018242432:error:1407742E:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version:../deps/openssl/openssl/ssl/s23_clnt.c:772:

💥 ###

PR Preview failed to build. (Last tried on Mar 16, 2020, 10:33 PM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

🚨 HTML Diff Service - The HTML Diff Service is used to create HTML diffs of the spec changes suggested in a pull request.

🔗 Related URL

If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.

@kainino0x kainino0x requested a review from kvark March 13, 2020 03:40
@kainino0x kainino0x force-pushed the synchronization-nits branch from 685694c to 57aebe8 Compare March 13, 2020 03:52
@kainino0x kainino0x force-pushed the synchronization-nits branch from 29e7f13 to 2404aac Compare March 14, 2020 01:19
@kvark
Copy link
Contributor

kvark commented Mar 16, 2020

Needs a rebase. Also, @JusSn want to have a look?

@kainino0x kainino0x requested a review from JusSn March 16, 2020 22:06
@kainino0x
Copy link
Contributor Author

ok, now it's merged properly.

Copy link
Contributor

@JusSn JusSn left a comment

Choose a reason for hiding this comment

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

Some suggestions!

@kainino0x kainino0x merged commit 041fd02 into gpuweb:master Mar 16, 2020
@kainino0x kainino0x deleted the synchronization-nits branch March 16, 2020 23:17
JusSn pushed a commit to JusSn/gpuweb that referenced this pull request Jun 8, 2020
JusSn pushed a commit to JusSn/gpuweb that referenced this pull request Jun 8, 2020
ben-clayton added a commit to ben-clayton/gpuweb that referenced this pull request Sep 6, 2022
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.

3 participants