Skip to content

Commit

Permalink
Merge pull request #340 from SysBioChalmers/develop
Browse files Browse the repository at this point in the history
GECKO 3.1.1
  • Loading branch information
edkerk authored Jul 17, 2023
2 parents 6378555 + f29852a commit dda5400
Show file tree
Hide file tree
Showing 43 changed files with 1,909 additions and 1,472 deletions.
8 changes: 5 additions & 3 deletions .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,7 @@ The following points should be considered when merging branches to `develop`:

* Make sure the branch gets accepted by at least one developer with writing access.
* Wait at least a day before merging, to allow other developers to inspect the pull request.
* To simplify the git history, use "Squash and merge" in pull requests to `develop` (but **never** use this option when merging to `main`, see also below).
* As soon as the branch is merged, confirm that `develop` is still possible to merge to `main` (this can be checked [here](https://github.com/SysBioChalmers/GECKO/compare/develop)). If conflicts appear, fix the conflict _locally_ as soon as possible in `develop` and then push it (note, **DO NOT** pull any other changes from `main` to `develop`, just the single file that is creating the conflict).

### Releasing a new version
Expand All @@ -169,8 +170,9 @@ When releasing, please follow these steps:
- Specify the intended version in the title, e.g. `GECKO 3.0.1`
- Indicating all new features/fixes/etc. and referencing every previous pull request included (examples [here](https://github.com/SysBioChalmers/GECKO/releases)).
- If any [issue](https://github.com/SysBioChalmers/GECKO/issues) gets solved in the release, write in the pull request description "Closes #X", where "X" is the issue number. That way the issue will be automatically closed after merge.
2. Wait at least a day for at least one approval. The GitHub Actions must also pass successfully.
3. Merge the pull request from `develop` to `main`.
4. Make the [new release](https://github.com/SysBioChalmers/GECKO/releases/new) at GitHub:
2. Wait at least a day for at least one approval. The GitHub Actions must also pass successfully.
3. Merge the pull request from `develop` to `main` with the option "Create a merge commit", **not** "Squash and merge".
4. Edit the `version.txt` in the `main` branch directly [link](https://github.com/SysBioChalmers/GECKO/edit/main/version.txt) to refer to the new version number. Commit the change directly to the `main` branch.
5. Make the [new release](https://github.com/SysBioChalmers/GECKO/releases/new) at GitHub:
- Define a new tag with the version, prefixed by `v`: e.g. `v3.0.1`.
- Copy the description from the corresponding PR from step 1.
20 changes: 12 additions & 8 deletions .github/ISSUE_TEMPLATE/bug-detected.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
name: Bug detected
about: GECKO is not working as it should...
about: Report a bug if GECKO was functioning as it should.
title: Bug detected
labels: bug
assignees: ''
Expand All @@ -13,18 +13,22 @@ assignees: ''
- What have you tried yourself to fix it? -->

#### Reproducing these results:
<!-- If applicable, please attach the problematic code. -->
<!-- If applicable, please attach the problematic code. This
is very useful for us to replicate the problem you encountered. -->
```matlab
% If you worked on matlab, paste your code here
```

#### System information
<!-- Please report details of the system where you encountered the bug. -->
* Operating system: <!-- (Windows/Mac/Linux; include version) -->
* Operating system: <!-- Windows/Mac/Linux; include version -->
* MATLAB version: <!-- E.g. 2022b -->
* GECKO version: <!-- If you did not use the latest GECKO version, have you tried to update and encountered the same problem? -->
* GECKO version: <!-- Have you tried the latest GECKO version? -->
* RAVEN version: <!-- Output from checkInstallation -->
* Solver: <!-- Gurobi (include version) / glpk / cobra / etc. -->

**I hereby confirm that I have:**
- [ ] Tested my code with [all requirements](https://github.com/SysBioChalmers/GECKO) for running GECKO
- [ ] Done this analysis in the `main` branch of the repository
- [ ] Checked that a similar issue does not exist already
#### I hereby confirm that:
<!-- Replace [ ] with [X] to check the box:-->
- [ ] My GECKO installation met [all requirements](https://github.com/SysBioChalmers/GECKO#required-software).
- [ ] This bug occurs in the `main` branch of the repository.
- [ ] A similar [issue](https://github.com/SysBioChalmers/GECKO/issues) does not already exist.
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ blank_issues_enabled: true
contact_links:
- name: Gitter Channel
url: https://gitter.im/SysBioChalmers/GECKO
about: Please ask and answer questions here.
about: You may also ask questions (or contribute with answers) at the GECKO Gitter channel.
10 changes: 5 additions & 5 deletions .github/ISSUE_TEMPLATE/new-feature.md
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
---
name: New feature
about: You would like GECKO to do something it does not yet do.
about: You can propose a new or modified features if you would like GECKO to do something it is not yet able to do.
title: New feature
labels: enhancement
assignees: ''

---

### Description of the new feature:
Expand All @@ -14,6 +13,7 @@ assignees: ''
- Do you have an idea on how to implement this feature?
- Will you implement this feature yourself (in that case, you can Assign yourself in the top right corner) or need help with this? We will gladly support you in this! -->

**I hereby confirm that I have:**
- [ ] Done this analysis in the `main` branch of the repository
- [ ] Checked that a similar issue does not exist already
#### I hereby confirm that:
<!-- Replace [ ] with [X] to check the box:-->
- [ ] The new feature is not already in the `main` branch of the repository.
- [ ] A similar [issue](https://github.com/SysBioChalmers/GECKO/issues) does not already exist.
17 changes: 11 additions & 6 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,14 @@
### Main improvements in this PR:
<!-- Pointwise mention what changes were made in what function. Examples:
- fix bug where `functionName` failed to do function X
- updated all documentation
- resolves issue #12345 -->
- Fixes:
- `functionName` failed to do function X
- Documentation:
- updated all documentation
- Features:
- Introduced option to do function Y (issue #12345) -->

**I hereby confirm that I have:**
<!-- Note: replace [ ] with [X] to check the box -->
- [ ] Selected `develop` as a target branch (top left drop-down menu)
#### Instructions on merging this PR:
<!-- Chose ONE of the following two options
and replace [ ] with [X] to check the box.-->
- [ ] This PR has `develop` as target branch, and will be resolved with a *squash-merge*.
- [ ] This PR has `main` as target branch, and will be resolved with a *merge commit*.
Loading

0 comments on commit dda5400

Please sign in to comment.