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

improvement: Ask to start http server if not running #7083

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tgodzik
Copy link
Contributor

@tgodzik tgodzik commented Jan 3, 2025

This will show up after indexing and users can start the server themselves or ignore it for the session.

Fixes #7072

This will show up after indexing and users can start the server themselves or ignore it for the session.

Should help with scalameta#7072
@tgodzik tgodzik requested a review from kasiaMarek January 3, 2025 13:30
@tgodzik
Copy link
Contributor Author

tgodzik commented Jan 3, 2025

I am still wondering if this will be more useful or whether it might be annoying to people 🤔

@tgodzik tgodzik force-pushed the fix-doctor-http-start branch from 55b4214 to 3794b0b Compare January 24, 2025 19:36
@tgodzik tgodzik marked this pull request as ready for review January 24, 2025 19:36
@tgodzik
Copy link
Contributor Author

tgodzik commented Jan 24, 2025

I think it's harmless enough improvement, so we can merge that. What do you think @kasiaMarek ?

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.

"WARN: Unable to run doctor. Make sure isHttpEnabled is set to true." and it is impossible to set to true
1 participant