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

fix: templates for GitHub Issues and Pull Requests #34

Merged
merged 5 commits into from
Mar 6, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 10 additions & 12 deletions .github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,25 +1,23 @@
### Description of the issue:
*Try to be as clear as possible: Is it something wrong/missing in the model? Is it a simulation issue? PLEASE DELETE THIS LINE.*

<!-- Try to be as clear as possible: is it something wrong/missing in the model, or is it an improvement suggestion? -->

#### Expected feature/value/output:
*How the reaction/metabolite/gene/simulation result should look (cite literature if needed). PLEASE DELETE THIS LINE.*

<!-- Please describe how the reaction/metabolite/gene/simulation result should like. It is helpful if literature can be referenced. -->

#### Current feature/value/output:
*How the reaction/metabolite/gene/simulation actually looks in the `main` branch. PLEASE DELETE THIS LINE.*

<!-- How the reaction/metabolite/gene/simulation actually looks in the `main` branch. -->

#### Reproducing these results:
*Please attach any code used below (if it's python code replace the keyword "matlab" with "python". PLEASE DELETE THIS LINE.*
<!-- Please attach any code used below (if it's python code replace the keyword "matlab" with "python". -->

```matlab

```

**I hereby confirm that I have:**
- [ ] Tested my code with [all requirements](https://github.com/SysBioChalmers/yeast-GEM#required-software---user) for running the model
- [ ] Done this analysis in the `main` branch of the repository
- [ ] Checked that a similar issue does not exist already
- [ ] If needed, asked first in the [Gitter chat room](https://gitter.im/SysBioChalmers/yeast-GEM) about the issue
<!-- Note: replace [ ] with [X] to check the box. -->

*Note: replace [ ] with [X] to check the box. PLEASE DELETE THIS LINE*
- [ ] tested my code with to run the model
- [ ] done this analysis in the `main` branch of the repository
- [ ] checked that a similar issue does not exist already
- [ ] if needed, asked first in the Gitter chat room about the issue
9 changes: 4 additions & 5 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
### Main improvements in this PR:
*Try to be as clear as possible: Is it fixing/adding something in the model? Is it an additional test/function/dataset? PLEASE DELETE THIS LINE.*
<!-- Try to be as clear as possible: Is it fixing/adding something in the model? Is it an additional test/function/dataset? -->

**I hereby confirm that I have:**
<!-- Note: replace [ ] with [X] to check the box. -->

- [ ] Tested my code on my own computer for running the model
- [ ] Selected `devel` as a target branch

*Note: replace [ ] with [X] to check the box. PLEASE DELETE THIS LINE*
- [ ] tested my code on my own computer for running the model
- [ ] selected `develop` as a target branch