-
Notifications
You must be signed in to change notification settings - Fork 292
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
Improve issue templates #3367
base: master
Are you sure you want to change the base?
Improve issue templates #3367
Conversation
Build successful. Some useful links:
This preview will be removed when the branch is merged. |
Sounds like it covers the most recurring topics 👍 Given that a large percentage does not replace the anthology ID in the template, I am only moderately optimistic regarding the impact, let's merge and prove me wrong. |
Do you mean that as a positive review? It still says Review required. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved but note small suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be useful to link the submitter to our documentation on this: https://github.com/acl-org/acl-anthology/wiki/Name-Variants
@nschneid yes, I meant that in a positive way. I did not want to do a formal review as others are more involved in handling day to day tickets and I would like to defer to them for an opinion. |
Some improvements to the Github issue templates based on some recurring issues I’ve seen.
@mjpost @anthology-assist Feel free to provide input based on what you think would be helpful!