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

Complete hls-hlint-plugin package metadata #1180

Merged
merged 4 commits into from
Jan 10, 2021

Conversation

jneira
Copy link
Member

@jneira jneira commented Jan 9, 2021

Maybe we should update the author field (and others)
//cc @alanz @ndmitchell

maintainer: [email protected] (for now)
maintainer: [email protected]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't you be listed as the maintainer too @jneira ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

We probably need some sort of generic reference to the hls team. Is that possible?

Copy link
Member Author

Choose a reason for hiding this comment

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

I would prefer some collective thing too, but i am not against include myself if needed, of course.
I guess we would need a generic email account.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe just put the github repo there?

Copy link
Member Author

Choose a reason for hiding this comment

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

mmm, that could work

@jneira jneira force-pushed the prepare-hlint-plugin-release branch from b88dcb8 to 1a7205a Compare January 9, 2021 18:57
@@ -2,11 +2,11 @@ cabal-version: 2.2
name: hls-hlint-plugin
version: 0.1.0.0
synopsis: Hlint integration plugin with Haskell Language Server
description: Please see README.md
description: Please see Haskell Language Server Readme (https://github.com/haskell/haskell-language-server#readme)
license: Apache-2.0
license-file: LICENSE
author: Many,TBD when we release
Copy link
Member Author

Choose a reason for hiding this comment

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

I guess we should to change this one for sure 😄
If we got a collective description, we could use it here

@jneira jneira force-pushed the prepare-hlint-plugin-release branch from 25f0521 to 4fe61fa Compare January 9, 2021 21:31
@jneira jneira added the merge me Label to trigger pull request merge label Jan 9, 2021
@mergify mergify bot merged commit 28d24d5 into haskell:master Jan 10, 2021
@jneira jneira deleted the prepare-hlint-plugin-release branch January 15, 2021 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Label to trigger pull request merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants