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 Haskell to LanguageKind #1421

Merged
merged 2 commits into from Feb 13, 2024

Conversation

michaelpj
Copy link
Contributor

As a maintainer of the Haskell language server, I would like to be able to name my language :D

As a maintainer of the Haskell language server, I would like to be able
to name my language :D
Copy link
Member

@dbaeumer dbaeumer left a comment

Choose a reason for hiding this comment

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

See last comment

@@ -1929,6 +1929,7 @@ export namespace LanguageKind {
export const Go = 'go' as const;
export const Groovy = 'groovy' as const;
export const Handlebars = 'handlebars' as const;
export const Haskell = 'haskell' as const;
Copy link
Member

Choose a reason for hiding this comment

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

Needs @SInCE and @proposed. See D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The whole enum has @since 3.18 and @proposed, so are the individual entries not all implicitly the same? It seems a bit odd that only a few of the entries have those annotations: shouldn't all of them have it or none of them?

Copy link
Member

Choose a reason for hiding this comment

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

Good point.

@dbaeumer
Copy link
Member

dbaeumer commented Feb 8, 2024

We also need to update the specification. Can you provide a PR for this as well?

@michaelpj
Copy link
Contributor Author

michaelpj commented Feb 8, 2024

Sure.

@michaelpj
Copy link
Contributor Author

The current version of the 3.18 spec doesn't have LanguageKind in it at all yet.

@dbaeumer
Copy link
Member

dbaeumer commented Feb 9, 2024

@michaelpj
Copy link
Contributor Author

Got it: PR here: microsoft/language-server-protocol#1898

@dbaeumer dbaeumer enabled auto-merge (squash) February 13, 2024 11:24
@VSCodeTriageBot VSCodeTriageBot added this to the February 2024 milestone Feb 13, 2024
@dbaeumer dbaeumer merged commit 21f4f0a into microsoft:main Feb 13, 2024
6 checks passed
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.

None yet

4 participants