Skip to content

Update CONTRIBUTING to match reality and add issue templates for bug reports, feature requests, and usage questions #1644

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

Merged
merged 5 commits into from
Jul 10, 2019

Conversation

0b10011
Copy link
Contributor

@0b10011 0b10011 commented Jul 1, 2019

Description

CONTRIBUTING.md was pretty out of date and didn't include any information about cargo test or cargo fix. There were also some grammar/wording issues that I fixed along the way.

Additionally, I added a section for getting started with hacking on PHPWord, which may not be completely obvious to many people, especially those using GitHub, Composer, or PHPUnit for the first time.

In the process, I noticed the existing singular issue template was a bit confusing and was trying to do many things at once (bug report, feature request, and prevent usage questions). I replaced that with three issue templates using GitHub's new functionality, one for each of those use cases:

  • .github/ISSUE_TEMPLATE/bug_report.md will be labeled "Bug report: Create a report to help improve PHPWord", label the issue with "Bug Report", and ask users to describe the bug, code to reproduce, expected and current behavior, and context. I tried to match the wording of the current issue template for the relevant sections.
  • .github/ISSUE_TEMPLATE/feature_request.md will be labeled "Feature request: Suggest an idea for this project", label the issue "Change Request", and ask the users to describe the problem they're trying to address, the solution they'd like, alternatives they've thought of, and additional context (I've mostly matched GitHub's default for this)
  • .github/ISSUE_TEMPLATE/how-to-use.md will be labeled "How to Use PHPWord: Find out how to use PHPWord", label any issues created with "WontFix", and tell the user not to use the issue tracker to ask how to use PHPWord, along with links to documentation, sample code, and Stack Overflow to ask questions.

I also moved the pull request template to /.github/ to avoid confusion for anyone looking to edit the templates (better to have everything in one spot).

I tried to stay true to the current wording/format/etc, but may have strayed in some areas. Feel free to update as necessary, or I'll be around if you need me to make any changes.

Fixes # (issue)

Checklist:

  • I have run composer run-script check --timeout=0 and no errors were reported
  • The new code is covered by unit tests (check build/coverage for coverage report)
  • I have updated the documentation to describe the changes

@coveralls
Copy link

coveralls commented Jul 1, 2019

Coverage Status

Coverage remained the same at 94.63% when pulling b13aa70 on 0b10011:contributing into fe531ca on PHPOffice:develop.

@troosan troosan added this to the v0.17.0 milestone Jul 10, 2019
@troosan troosan merged commit 58adc1e into PHPOffice:develop Jul 10, 2019
@0b10011 0b10011 deleted the contributing branch July 11, 2019 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants