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

[skip circleci] Add modern issue templates #2394

Merged
merged 13 commits into from Nov 29, 2021
Merged

[skip circleci] Add modern issue templates #2394

merged 13 commits into from Nov 29, 2021

Conversation

@jneira jneira changed the title Add modern issue templates [skip circleci] Add modern issue templates Nov 24, 2021
Copy link
Collaborator

@fendor fendor left a comment

Choose a reason for hiding this comment

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

Awesome, thank you! LGTM

.github/ISSUE_TEMPLATE/bug_report.md Outdated Show resolved Hide resolved
Co-authored-by: fendor <fendor@users.noreply.github.com>
Copy link
Collaborator

@michaelpj michaelpj left a comment

Choose a reason for hiding this comment

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

If you want to get really fancy, you could use the new issue forms: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms

My main thought is that I think as much as possible we should enhance the troubleshooting page and direct people there, so the information is centralised in one place. I'm hoping to make a PR for some improvements there in a bit!

.github/ISSUE_TEMPLATE/blank-issue.md Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.md Outdated Show resolved Hide resolved
-->

### Your environment
Output of `haskell-language-server --probe-tools` or `haskell-language-server-wrapper --probe-tools`:
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is still useful, no?

Copy link
Collaborator

Choose a reason for hiding this comment

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

haskell-language-server --debug . also prints the tools, so I think this is indeed superfluous.

Execute in the root of your project the command `haskell-language-server --debug .` and paste the logs here:

Execute in the root of your project the command `haskell-language-server-wrapper --debug .` and paste the logs here:
(if you are using the vscode extension check the executable location [here](https://github.com/haskell/vscode-haskell#downloaded-binaries))
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if we should just write more about this on the troubleshooting page, and then here we could say "If you don't know where to find the haskell-language-server exectuable, consult the troubleshooting page"

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, although a direct reference to the location may be helpful too? I mean, we are linking the troubleshooting page in the beginning but in this point the focus is locate it and the table linked gives you the path

@@ -42,7 +44,7 @@ Debug output:
```
</details>

Paste the logs from the lsp-client, e.g. for [VS Code](https://github.com/haskell/vscode-haskell#troubleshooting)
Paste the logs from the lsp-client, you can check instructions about for VS Code [here](https://github.com/haskell/vscode-haskell#troubleshooting)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Again, I think we should refer people to the relevant section of the troubleshooting page.

Copy link
Member Author

Choose a reason for hiding this comment

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

hmm https://github.com/haskell/vscode-haskell#troubleshooting would not be the relevant page?

Co-authored-by: Michael Peyton Jones <me@michaelpj.com>
@jneira
Copy link
Member Author

jneira commented Nov 24, 2021

If you want to get really fancy, you could use the new issue forms: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms

wow didnt know about that feature, looks really nice. I think i will give a try

Co-Authored-by @michaelpj
@jneira
Copy link
Member Author

jneira commented Nov 24, 2021

My main thought is that I think as much as possible we should enhance the troubleshooting page and direct people there, so the information is centralised in one place. I'm hoping to make a PR for some improvements there in a bit!

what do you think revisit the issue template after improving the troubleshooting section and update the links?

@michaelpj
Copy link
Collaborator

what do you think revisit the issue template after improving the troubleshooting section and update the links?

Yes, fine!

@jneira
Copy link
Member Author

jneira commented Nov 24, 2021

wow didnt know about that feature, looks really nice. I think i will give a try

It looks nice but it will take some time to get it right so i think is better to improve content and switch to forms in other pr

@jneira jneira added the merge me Label to trigger pull request merge label Nov 28, 2021
@jneira jneira added merge me Label to trigger pull request merge and removed merge me Label to trigger pull request merge labels Nov 28, 2021
@mergify mergify bot merged commit 268ef19 into master Nov 29, 2021
drsooch pushed a commit to drsooch/haskell-language-server that referenced this pull request Dec 3, 2021
* Add modern issue templates

* Replace with modern issues templates

* Mention discussions in the blank issue

* Mention explicitly vscode binaries location

* Correct grammar

Co-authored-by: fendor <fendor@users.noreply.github.com>

* Correct grammar

Co-authored-by: Michael Peyton Jones <me@michaelpj.com>

* Rephrase into

Co-Authored-by @michaelpj

* they already let you open a blank one

* Comment out suggestions

Co-authored-by: fendor <fendor@users.noreply.github.com>
Co-authored-by: Michael Peyton Jones <me@michaelpj.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
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
3 participants