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 numeric-version option for wrapper and server #241

Merged
merged 1 commit into from
Jul 26, 2020

Conversation

fendor
Copy link
Collaborator

@fendor fendor commented Jul 26, 2020

closes #238

Also correctly shuts down the language server if --version was given.

Minor rework of argument parser.

This might cause breakage, since combinations such as haskell-language-server --version --lsp used to be valid but are not anymore. Not sure if and how important that is, though. I think we can change it if it is important, but we might lose the stronger types or increase the complexity of the arguments parser.

@fendor fendor force-pushed the numeric-version branch 2 times, most recently from 64dd393 to fa44b4d Compare July 26, 2020 15:54
@fendor fendor requested review from lukel97, alanz and jneira and removed request for lukel97 July 26, 2020 15:54
@fendor
Copy link
Collaborator Author

fendor commented Jul 26, 2020

@hasufell Should the --numeric-version have a newline at the end?

@hasufell
Copy link
Member

Newline or not will both work

exe/Wrapper.hs Outdated Show resolved Hide resolved
Also correctly shuts down the language server if `--version`
was given.

Minor rework of argument parser.
Copy link
Member

@Ailrun Ailrun left a comment

Choose a reason for hiding this comment

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

LGTM :)

@lukel97 lukel97 merged commit f2384e1 into haskell:master Jul 26, 2020
pepeiborra pushed a commit that referenced this pull request Dec 27, 2020
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.

hls --version does not work
4 participants