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

Add support for bloop server #400

Merged
merged 5 commits into from
Mar 28, 2018
Merged

Conversation

jvican
Copy link
Contributor

@jvican jvican commented Mar 27, 2018

Adds improvements in nailgun that fix #320.

@jvican jvican added enhancement nailgun ergonomics Any change that affects developer ergonomics and the easiness of use of bloop. labels Mar 27, 2018
@jvican jvican force-pushed the topic/sync-nailgun-server-support branch 2 times, most recently from f5819be to aa82ce8 Compare March 27, 2018 15:10
@jvican jvican force-pushed the topic/sync-nailgun-server-support branch from aa82ce8 to 53de32c Compare March 28, 2018 07:40
@jvican
Copy link
Contributor Author

jvican commented Mar 28, 2018

CI error in 53de32c is spurious.

Copy link
Collaborator

@Duhemm Duhemm left a comment

Choose a reason for hiding this comment

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

Maybe that we should also use bloop server to start the service in the homebrew formula?

The server is installed by default in the same directory than the `bloop`
client, so `bloop server` will try to find the location of the server in the
same directory where `bloop` is installed. If this is not the case, run `bloop
---server-location=/foo/my-blp-server server` instead.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are there 3 - on purpose?


The bloop server can serve responses of many concurrent clients.

#### Example
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that we should get rid of this paragraph. It doesn't bring much, and the given example is not a runnable command (no project is specified).

@jvican
Copy link
Contributor Author

jvican commented Mar 28, 2018

Maybe that we should also use bloop server to start the service in the homebrew formula?

bloop server is meant to be used by the users, so I don't see much value in doing that.

Feedback addressed.

@jvican jvican merged commit 23e62f9 into master Mar 28, 2018
@tgodzik tgodzik deleted the topic/sync-nailgun-server-support branch September 7, 2021 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ergonomics Any change that affects developer ergonomics and the easiness of use of bloop. nailgun
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename blp-server to bloop-server
2 participants