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

Move emojis in issue templates to the titles #1058

Merged
merged 1 commit into from
Aug 27, 2022

Conversation

hoechenberger
Copy link
Member

Followup to #1056

I think it has a much better effect this way!

Merge checklist

Maintainer, please confirm the following before merging.
If applicable:

  • All comments are resolved
  • This is not your own PR
  • All CIs are happy
  • PR title starts with [MRG]
  • whats_new.rst is updated
  • New contributors have been added to CITATION.cff
  • PR description includes phrase "closes <#issue-number>"

Followup to mne-tools#1056

I think it has a much better effect this way!
@hoechenberger hoechenberger merged commit 7eb37ef into mne-tools:main Aug 27, 2022
@hoechenberger hoechenberger deleted the issue-template-emojis branch August 27, 2022 08:39
@hoechenberger
Copy link
Member Author

hoechenberger commented Aug 27, 2022

@hoechenberger
Copy link
Member Author

@sappelhoff Can we change the order of the items? I'd like to put the forum first, followed by feature request, bug report, and BIDS spec. Just so as to emphasize that users should probably ask on the forum before opening an issue here

@sappelhoff
Copy link
Member

Looks better, thanks!

No idea about the order. Perhaps in the config yml?

@hoechenberger
Copy link
Member Author

It's not urgent, was just an idea :)

@drammock
Copy link
Member

IIRC you can't make the forum link come before the entries that are actual issue templates. I looked into it a few months ago because I had the same instinct that it would be better to have that at the top

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.

3 participants