Update CONTRIBUTING.md

This commit is contained in:
vimux 2019-10-31 11:02:39 -04:00
parent 8ebd83fc14
commit 265e8be1f2
No known key found for this signature in database
GPG key ID: 5A34FDC4EE832E34

View file

@ -1,45 +1,105 @@
# Contributing to Binario # Contributing to Binario
Looking to contribute something to **Binario** theme? Bug reports and code patches are welcome. **Binario** welcomes contributions and corrections. Before contributing, please make sure you have read the guidelines
below. If you're a newcomer to open source and you haven't contributed to other projects or used
Please take a moment to review this document in order to make the contribution process easy and effective for everyone [Git](https://git-scm.com/) before, you should make yourself familiar before proceeding.
involved.
## Issues ## Issues
The [issue tracker](https://github.com/vimux/binario/issues) is the preferred channel for bug reports and features The [issue tracker](https://github.com/vimux/binario/issues) is the preferred channel for bug reports and features
requests, but please respect the following restrictions: requests, but please respect the following restrictions:
* Please **do not** combine a few problems or features in one issue. Create separate issues if needed. **General requirements**
* Please **do not** create issue that contains title only. Write useful title and description.
* Please **do not** post comments consisting solely of "+1" or emoji. Use
[GitHub's "reactions" feature](https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments) instead.
The project maintainer reserve the right to delete such comments.
* Please specify the exact steps to reproduce bug.
## Pull requests (PR) * Issue must be written in English.
* Please **do not** combine a few problems or feature requests in one issue. Create separate issues if needed.
* Please **do not** create an issue that contains only title. Write a clear title and useful description.
* Please **do not** use the issue tracker for personal support requests.
* Please **do not** post comments consisting solely of "+1" or emoji. The project maintainer reserve the right to delete
such comments. Use
[GitHub's reactions feature](https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments) instead.
* Search first before filing a new issue. Please check existing open or recently closed issues to make sure somebody
else hasn't already reported the issue.
**Reporting bugs**
When creating a new bug issue make sure to include the following information:
* Your environment e.g. OS version, Hugo version, theme is up to date? Anything unusual about your environment or
deployment.
* Specify the exact steps to reproduce the bug in as many details as possible with code examples. Include links to files
or demo projects, or copy/pasteable snippets, which you use in those examples.
* Any message or error you get from Hugo, if you do.
* A screenshot of any visual bug.
**Note:** If you find a **Closed** issue that seems like it is the same bug that you're experiencing, open a new issue
and include a link to the original issue in the body of your new one.
**Proposing features**
* Explain the proposed feature, what it should do, why it is useful, and alternatives considered if possible. Please
note that the project maintainer may close this issue or ask you to create a Pull Request if this is not something that
the project maintainer sees as a sufficiently high priority.
Following these guidelines helps maintainer and the community understand your suggestion and find related suggestions.
## Pull Requests (PR)
**Please ask first** before embarking on any significant pull request (e.g. implementing features or refactoring code), **Please ask first** before embarking on any significant pull request (e.g. implementing features or refactoring code),
otherwise you risk spending a lot of time working on something that the project maintainer might not want to merge into otherwise, you risk spending a lot of time working on something that the project maintainer might not want to merge into
the project. the project.
**IMPORTANT: By submitting a patch, you agree to allow the project owner to license your work under the terms of the Please respect our Pull Request Acceptance Criteria. For larger changes, you will likely receive multiple rounds of
[MIT License](LICENSE).** comments and it may take some time to complete.
Please respect coding guidelines used throughout the project and our PR acceptance criteria. ### Pull Request Acceptance Criteria
### PR acceptance criteria * Keep the change in a single PR as small as possible
* Please keep the change in a single PR as small as possible.
* 1 PR = 1 FIX or FEATURE (do not combine things, send separate PR if needed) * 1 PR = 1 FIX or FEATURE (do not combine things, send separate PR if needed)
* Does not break any existing functionality * PR with irrelevant changes won't be merged. If you do want to fix formatting or style, do that in a separate PR
* Your changes can merge without problems * Use a clear and descriptive branch name (e.g. **NOT** "patch-1" or "update")
* Don't create a Pull Request from master branch
* Provide a reasonable PR title and description * Provide a reasonable PR title and description
* PR must be written in English
* If the PR changes the UI it should include before-and-after screenshots or a link to a video * If the PR changes the UI it should include before-and-after screenshots or a link to a video
* Be prepared to answer questions and make changes in your code * Keep PR up to date with upstream/master
* Pay attention to any automated CI failures reported in the Pull Request
* PR solves a common use case that several users will need in their real-life projects, not only your specific problems
* If you've added or modify SVG, ensure that each SVG file:
* Be less than 2048 bytes
* Be minified to a single line with no formatting
* Not contain any JS or CSS section inside it
* Not contain any additional transformations (matrix, translate, scale)
* Сompatible with [MIT License](LICENSE.md)
* Maintain clean commit history and use meaningful commit messages. Pull Requests with messy commit history (with
commit messages like "update", "another update", etc) are difficult to review and won't be merged, even if the changes
are good enough
* Be prepared to answer questions and make code changes. The project maintainer expect you to be reasonably responsive
to those feedback, otherwise the PR will be closed after 2-4 weeks of inactivity
**IMPORTANT: No guarantees can be made that your pull request will be accepted.** ### Pull Request Contribution Prerequisites
* You have Node & npm installed
* You have Hugo installed at v0.48.0+
* You are familiar with Git
### Pull Request Process
1. Fork the repository
1. Clone down the repository to your local system
1. Run `npm i` in the repository root
1. Create a new *dedicated branch* with descriptive name from `master`
1. Make your change and commit to the new branch from the previous step
1. Write a clear commit message
1. If you've added code that need documentation, update the README.md
1. Make sure your code lints (`npm test`)
1. Push to your fork
1. Submit a Pull Request (PR) to the upstream
---
**⚠️ IMPORTANT: No guarantees can be made that your pull request will be accepted.**
## License ## License
By contributing your code, you agree to license your contribution under the [MIT License](LICENSE). By contributing to Binario, you agree that your contributions will be licensed under [MIT License](LICENSE.md).