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

JEP draft: WebSocket Services #250

Merged
merged 12 commits into from
Dec 4, 2019

Conversation

jglick
Copy link
Member

@jglick jglick commented Nov 22, 2019

Prototype implementation in jenkinsci/jenkins#4369 and its upstreams.

Copy link
Member

@rtyler rtyler left a comment

Choose a reason for hiding this comment

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

From my perspective this a well structured JEP, great work @jglick! I would like one other person to weigh in, but if they do not before the beginning of December, I'll go ahead and number this (assuming no specific range requested), .

@jglick jglick marked this pull request as ready for review November 22, 2019 22:26
jep/0000/README.adoc Outdated Show resolved Hide resolved
Copy link
Member

@omehegan omehegan left a comment

Choose a reason for hiding this comment

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

From the perspective of a casual reader, this is excellent. It's thorough and easy to understand, and I'm not left with any questions about the proposal. Well done!

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

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

Looks good from the structure perspective. +1 for accepting it as a JEP draft

Copy link
Contributor

@jeffret-b jeffret-b left a comment

Choose a reason for hiding this comment

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

This is great. It explains things well and provides useful information. The approach makes a lot of sense. I like how it fits into the existing infrastructure. This will be a great addition to the existing infrastructure mechanisms.

I haven't yet looked at the implementation PRs, but I'll glance over them now and plan on more thorough reviews later.

jep/0000/README.adoc Outdated Show resolved Hide resolved
@jglick
Copy link
Member Author

jglick commented Dec 2, 2019

@rtyler I guess this can be merged & numbered now?

@oleg-nenashev
Copy link
Member

oleg-nenashev commented Dec 2, 2019 via email

@bitwiseman
Copy link
Contributor

@oleg-nenashev No reason to wait.

@bitwiseman bitwiseman merged commit e4fc9fc into jenkinsci:master Dec 4, 2019
@oleg-nenashev
Copy link
Member

oleg-nenashev commented Dec 4, 2019 via email

@jglick jglick deleted the jep-submission-websocket branch December 4, 2019 16:42
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.

8 participants