Skip to content

Commit

Permalink
Add templates for GitHub issues and pull requests
Browse files Browse the repository at this point in the history
  • Loading branch information
snukky authored Mar 17, 2020
1 parent adba021 commit 9ccb075
Show file tree
Hide file tree
Showing 5 changed files with 81 additions and 21 deletions.
21 changes: 21 additions & 0 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
---
name: Bug report
about: Create a report to help us improve
title: ''
labels: bug
assignees: ''

---

### Bug description
Please add a clear and concise description of the bug, including observed and if possible expected behavior.

### How to reproduce
Describe steps or include command to reproduce the behavior.

### Context
* Marian version: Paste the output of `--version` here
* CMake command: Type the cmake command you used and attach the output of `--build-info all`
* Log file: Attach your training/decoding logs

Add any other information about the problem here.
1 change: 1 addition & 0 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
blank_issues_enabled: true
15 changes: 15 additions & 0 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
---
name: Feature request
about: Suggest an idea for this project
title: ''
labels: enhancement
assignees: ''

---

### Feature description
Please add a concise description of what the problem is and describe the solution you would like to see.
Add links to a paper, another toolkit, etc. if relevant.

### Example
Add a usage examples for the new feature, e.g. a command line.
25 changes: 25 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
### Description
Please add a clear and concise description of the changes.

This PR fixes a bug/adds a new feature/refactorizes the code/does something else.
It is related to issues: #998, #999, ...

List of changes:
- ...
- ...
- ...

Added dependencies: none

### How to test
Describe how to test your changes, adding command line examples and sample input/output files if relevant.
Point to unit tests or regression tests covering the changes if they have been added.

Describe how you have tested your code, including OS and the cmake command.

### Checklist

- [ ] I have tested the code manually
- [ ] I have run regression tests
- [ ] I have read and followed CONTRIBUTING.md
- [ ] I have updated CHANGELOG.md
40 changes: 19 additions & 21 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,12 @@ How to contribute to Marian

Did you find a bug or want to ask a question? Great!

* Ensure the bug was not already reported or the question was not asked
* For bugs check the [github
issues](https://github.com/marian-nmt/marian-dev/issues)
* For answers to your question search posts on the [Google discussion
group](https://groups.google.com/forum/#!forum/marian-nmt)
* Open a new issue/question if you're unable to find yours
* Ensure the bug was not already reported or the question was not asked on
[the github issues](https://github.com/marian-nmt/marian-dev/issues) or
[the Google discussion group](https://groups.google.com/forum/#!forum/marian-nmt).
* Open a new issue/question if you're unable to find yours.
* For bugs please provide as much relevant information as possible, and do not
forget to paste a log
forget to attach training/decoding logs and your Marian command.

You are also welcome to request a new feature.
Especially if you plan to help us adding it :)
Expand All @@ -22,19 +20,20 @@ Especially if you plan to help us adding it :)
## Submitting changes

Whenever possible, please send a Github Pull Request with a clear list of what
you've done. Feel free to also update CHANGELOG.md file. We will love you
you've done. Feel free to update CHANGELOG.md file. We will love you
forever if you provide unit or regression tests.

Please follow our coding convention (below) and make sure all of your commits
are atomic (learn more about _git squash_ to merge multiple commits and _git
rebase -i_ to split a single huge commit into smaller pieces).
Please follow our coding convention (below) and do not forget to test your
changes by running unit tests:

Ideally test your changes by running [Marian regression
tests](http://github.com/marian-nmt/marian-regression-tests.git) locally:
cd marian-dev/build
cmake .. -DCOMPILE_TESTS=ON
make test

git clone http://github.com/marian-nmt/marian-regression-tests.git
cd marian-regression-tests.git
make BRANCH=<your_branch_name> install
and [regression tests](http://github.com/marian-nmt/marian-regression-tests.git):

cd marian-dev/regression-tests
make install
./run_mrt.sh


Expand All @@ -43,16 +42,15 @@ tests](http://github.com/marian-nmt/marian-regression-tests.git) locally:
Main code style rules:

* no tabs, 2 whitespaces instead
* lines no longer than 80 characters
* lines no longer than 100 characters
* no trailing whitespaces
* no space between control statements and opening brackets
* `UpperCamelCase` for class names
* `camelCaseWithTrailingUnderscore_` for class variables
* `camelCase` for variables, methods and functions
* `UPPERCASE_WITH_UNDERSCORES` for constants

Ideally, use the provided `.clang-format` file (in the root directory) for
[ClangFormat](https://clang.llvm.org/docs/ClangFormat.html) to format your code, e.g.

clang-format-3.8 <path_to_file>
You may also use [ClangFormat](https://clang.llvm.org/docs/ClangFormat.html)
and the `.clang-format` file provided in the root directory to help you with
code formatting.

0 comments on commit 9ccb075

Please sign in to comment.