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

GitHub issues + pull request templates #351

Merged
merged 11 commits into from
Jun 18, 2024
Merged

Conversation

nullsystem
Copy link
Collaborator

@nullsystem nullsystem commented Jun 8, 2024

  • NOTE: This was first done via GitHub's settings and some template to create the issues template. That's why the branch is within the main repo and why the issues markdown formatting is different to the PR's one (which doesn't have a template to create a template).
  • Add issues template giving an option between bug report or feature request
  • Structured especially for bug report so it can be followed through
  • Altered slightly from GitHub's given default to fit a bit more for NT;RE
  • resolves GitHub issues and pull requests templates #350
  • Issues reporting are both YAML files to leverage form feature. PR will be kept as markdown since it'd rather it be more free-formed.

@nullsystem nullsystem requested review from a team and blaberry June 8, 2024 22:50
@nullsystem nullsystem added the documentation Improvements or additions to documentation label Jun 8, 2024
@nullsystem nullsystem marked this pull request as draft June 8, 2024 23:09
@kassibuss
Copy link
Contributor

kassibuss commented Jun 8, 2024

LGTM Edit: Didnt noticce it was draft

* Add issues template giving an option between bug report or feature request
* Structured especially for bug report so it can be followed through
* Altered slightly from GitHub's given default to fit a bit more for NT;RE
* Add PR template

* resolves: #350
@nullsystem nullsystem marked this pull request as ready for review June 9, 2024 00:54
@nullsystem nullsystem changed the title GitHub issues template GitHub issues + pull request templates Jun 9, 2024
@nullsystem nullsystem requested a review from Rainyan June 9, 2024 01:22
Rainyan
Rainyan previously approved these changes Jun 9, 2024
@nullsystem
Copy link
Collaborator Author

Actually, should the bug reports be turned into using the "issue forms" yaml format instead? It does seem to give more controls, probably an advantage with the URL linking and put in build info + OS at specific places. Not a fan of the yaml format though.

@Rainyan
Copy link
Collaborator

Rainyan commented Jun 9, 2024

Actually, should the bug reports be turned into using the "issue forms" yaml format instead? It does seem to give more controls, probably an advantage with the URL linking and put in build info + OS at specific places. Not a fan of the yaml format though.

YAML comes up more prominently when looking for GitHub docs as to how to do this stuff. Might be a good idea.

@blaberry
Copy link
Collaborator

blaberry commented Jun 9, 2024 via email

@nullsystem
Copy link
Collaborator Author

nullsystem commented Jun 9, 2024

I mentioned it earlier, but one of the issues with issues not created through the projects view is that we lose the bounty tracking by default, though we could probably fix this relatively easily with some auto-tagging. Apologies for the surface level insight, I might have some time later today to give a more in depth review.

@blaberry It seems like creating an issue through projects is able to use the templates anyway. Clicking on "+" -> Create a new issue -> Above "Add a title" currently it says "Blank issue in NeotokyoRebuild/neo" but with templates put in it'll be some template instead.
Even then if an issue is created through the standard issue, it can be added to the projects stuff anyway.

EDIT: Actually it does seem like templates supports auto-adding to projects anyway.

@blaberry
Copy link
Collaborator

blaberry commented Jun 9, 2024

Yeah using the templates for project-created issues is a great thing, I was only concerned about the opposite flow, ie how do we make sure issues don't get lost by not being assigned a project/milestone. But if we can add them to the bounty project by default then all good by me.

@nullsystem
Copy link
Collaborator Author

nullsystem commented Jun 9, 2024

So regarding the URL, I'm wondering if it's a GitHub's bug but the URL GET parameters seemed to overridden the defaults set in the template. However these parameters can be set back anyway. Ending URL with: /issues/new?template=bug_report.yml&build=20240609_abc123&labels=bug&title=[BUG]&projects=NeoTokyoRebuild/1 for example will fill the build input box with 20240609_abc123, put in "bug" label, prefix title with "[BUG]", and auto-add it to the project board/list.

If someone just go through the normal issue creation workflow (via Issues page, not project board), these defaults parameters (bug label, project board auto-add, etc..) will be applied as expected. In which in this case creating an issue from the Project section isn't a necessity anymore to add it to Project board.

It seems like the markdown somehow doesn't recognize the projects section and can't auto-add to project, so will also change the feature request to YAML/form format also.

Rainyan
Rainyan previously approved these changes Jun 9, 2024
@Rainyan

This comment was marked as resolved.

@nullsystem
Copy link
Collaborator Author

nullsystem commented Jun 10, 2024

@Rainyan I kind of think bugs titles as "State of things currently" and features/changes titles as "The desired state of things". Either not a big deal or give a guideline for bugs title to be consistent.

Not taking Qt's bug reports as a guideline, but the general gist over there: https://bugreports.qt.io/browse/QTBUG-126211?jql=project%20%3D%20QTBUG
Are kinda like that bug vs features/changes split also
Other projects like EX: raddebugger or Hyprland points to that direction also

@nullsystem nullsystem added the New Changes for Reviewers New changes has been made. This label is mostly to notify reviewers. label Jun 14, 2024
Rainyan
Rainyan previously approved these changes Jun 14, 2024
blaberry
blaberry previously approved these changes Jun 17, 2024
@nullsystem
Copy link
Collaborator Author

@blaberry
https://github.com/NeotokyoRebuild/neo/blob/0466cdce4b24f7133ba538ad94a473113dfcb0d0/.github/ISSUE_TEMPLATE/feature_request.yml

There is a problem with this template

YAML syntax error: (): could not find expected ':' while scanning a simple key at line 40 column 7. Learn more about this error.

blaberry
blaberry previously approved these changes Jun 17, 2024
.github/ISSUE_TEMPLATE/feature_request.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/feature_request.yml Outdated Show resolved Hide resolved
Rainyan
Rainyan previously approved these changes Jun 17, 2024
@nullsystem nullsystem requested a review from blaberry June 17, 2024 20:46
@nullsystem nullsystem merged commit d90900f into master Jun 18, 2024
@nullsystem nullsystem deleted the GH-350_issuesTemplates branch June 18, 2024 23:14
Rainyan pushed a commit that referenced this pull request Jul 1, 2024
* GitHub issues + PR template

* Add issues template giving an option between bug report or feature request
* Structured especially for bug report so it can be followed through
* Altered slightly from GitHub's given default to fit a bit more for NT;RE
* Add PR template

* resolves: #350

* More comments to aid issue write ups

* PR review changes

* Update bug report to yml forms format

* Change feature_request to yaml

* Added more optional inputs, improve wordings and clarifications

* Add some more links and expand the feature requests a little.

* Fix weird newline breaking the parsing.

* PR review, title space, placeholder example feature req

* Five Ws

---------

Co-authored-by: Linn <hamstertejp@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation New Changes for Reviewers New changes has been made. This label is mostly to notify reviewers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub issues and pull requests templates
4 participants