Skip to content

Improve issue & pull-request templates#4266

Merged
bajtos merged 2 commits into
masterfrom
improve/github-templates
Oct 7, 2019
Merged

Improve issue & pull-request templates#4266
bajtos merged 2 commits into
masterfrom
improve/github-templates

Conversation

@bajtos

@bajtos bajtos commented Oct 4, 2019

Copy link
Copy Markdown
Member

Pull requests

  • Make it easier for contributors to mix the text provided by GitHub
    from the commit messages with the template content.

  • Rework the checklist to follow the style we use in loopback-next.

Issues

Introduce four different templates for different issue types:

  • Bug reports
  • Feature requests
  • Questions
  • Security

Make it easier for contributors to mix the text provided by GitHub
from the commit messages with the template content.

Rework the checklist to follow the style we use in loopback-next.

Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
@bajtos
bajtos requested a review from dhmlau October 4, 2019 08:53
@bajtos bajtos self-assigned this Oct 4, 2019
@bajtos bajtos added the doc label Oct 4, 2019

@dhmlau dhmlau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @bajtos for adding the templates. I have some suggestions in the review comments. Thanks.

Comment thread .github/ISSUE_TEMPLATE/Security_vulnerability.md Outdated
Comment thread .github/ISSUE_TEMPLATE/Feature_request.md
Comment thread .github/ISSUE_TEMPLATE/Bug_report.md
@bajtos

bajtos commented Oct 4, 2019

Copy link
Copy Markdown
Member Author

@dhmlau I added links to LB4, LGTY now?

@bajtos
bajtos requested a review from dhmlau October 4, 2019 13:37
@bajtos
bajtos force-pushed the improve/github-templates branch from e1e3176 to 87cd160 Compare October 4, 2019 14:06
@bajtos
bajtos force-pushed the improve/github-templates branch from 87cd160 to ff9c212 Compare October 4, 2019 14:23
Comment thread .github/ISSUE_TEMPLATE/Feature_request.md Outdated
Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants