-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
c8b51ac
commit de79c5d
Showing
3 changed files
with
94 additions
and
26 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,30 +1,11 @@ | ||
| <!--- Provide a general summary of your changes in the Title above --> | ||
| Please go the the `Preview` tab and select the appropriate sub-template: | ||
|
|
||
| ## Description | ||
| <!--- Describe your changes in detail --> | ||
| [**Contributor PR**](?expand=1&template=PR_Template_Contributor.md) | ||
|
|
||
| ## Motivation and Context | ||
| <!--- Why is this change required? What problem does it solve? --> | ||
| <!--- If it fixes an open issue, please link to the issue here. --> | ||
| <!--- If it full fills an open feature request, please link to the feature request here. --> | ||
| [Version Upgrade **(for Admins)**](?expand=1&template=PR_Template_VersionUpgrade.md) | ||
|
|
||
| ## How has this been tested? | ||
| <!--- Please describe in detail how you tested your changes. --> | ||
| <!--- Include details of your testing environment, especially --> | ||
| <!--- the software you teste your change with. --> | ||
| <!--- Are other areas of the specification affected? --> | ||
|
|
||
| ## Snippets or Screenshots (if necessary): | ||
|
|
||
| ## Types of changes | ||
| <!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> | ||
| - [ ] Bug fix (non-breaking change which fixes an issue) | ||
| - [ ] New feature (non-breaking change which adds functionality) | ||
| - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) | ||
|
|
||
| ## Checklist: | ||
| <!--- Go over all the following points, and put an `x` in all the boxes that apply. --> | ||
| <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> | ||
| - [ ] My code follows the code style of this IG / specification. | ||
| - [ ] My change requires a change to the documentation or narrative (intend) of the IG. | ||
| - [ ] I have already updated the documentation / narrative (intend) accordingly. | ||
| <!--- If you have no idea, time, what so ever, you can also continue here without using a pull request template--> | ||
| <!--- Please keep in mind that if you use this short cut, your PR is lees likely to be merged, because of missing information.--> | ||
| ## Pull Request Short Cut | ||
| <!--- Describe your changes briefly. If you need some space to describe, please use the contributor template as mentioned at the top--> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
|
|
||
| # Contributor Pull Request | ||
| <!--- Provide a general summary of your changes in the Title above --> | ||
|
|
||
| ## Description | ||
| <!--- Describe your changes in detail --> | ||
|
|
||
| ## Motivation and Context | ||
| <!--- Why is this change required? What problem does it solve? --> | ||
| <!--- If it fixes an open issue, please link to the issue here. --> | ||
| <!--- If it full fills an open feature request, please link to the feature request here. --> | ||
|
|
||
| ## How has this been tested? | ||
| <!--- Please describe in detail how you tested your changes. --> | ||
| <!--- Include details of your testing environment, especially --> | ||
| <!--- the software you teste your change with. --> | ||
| <!--- Are other areas of the specification affected? --> | ||
|
|
||
| ## Snippets or Screenshots (if necessary): | ||
|
|
||
| ## Types of changes | ||
| <!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> | ||
| - [ ] Bug fix (non-breaking change which fixes an issue) | ||
| - [ ] New feature (non-breaking change which adds functionality) | ||
| - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) | ||
|
|
||
| ## Checklist: | ||
| <!--- Go over all the following points, and put an `x` in all the boxes that apply. --> | ||
| <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> | ||
| - [ ] My code follows the code style of this IG / specification. | ||
| - [ ] My change requires a change to the documentation or narrative (intend) of the IG. | ||
| - [ ] I have already updated the documentation / narrative (intend) accordingly. |
55 changes: 55 additions & 0 deletions
55
.github/PULL_REQUEST_TEMPLATE/PR_Template_VersionUpgrade.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,55 @@ | ||
| # Version Upgrade Template | ||
| <!--- parameters of your upgrade --> | ||
|
|
||
| Version: | ||
|
|
||
| Date: | ||
|
|
||
| ## Description | ||
| <!--- CAVEAT: You to be an admin to perform all the necessary jobs. --> | ||
| This is a Pullreuqest that requires an increase in the Version number. Therefore, multiple outside-github, related Task have to be performed and checked. | ||
|
|
||
| All jobs with an `x` in the boxes were performed to the best of knowledge. | ||
|
|
||
| ## Pre-Merge Activities | ||
| <!--- Lets check everything before we continue. --> | ||
| - [ ] This PR refers to a versioned Branch with a name and a version number in the form of N.n.n, e.g. "TC_3.2.1". | ||
| - [ ] This PR has a clean meaningful committ history. Minor committs or committs without descirption have been squashed, at the latest now. | ||
| - [ ] The _./github/workflows/main.yml_ refers to the correct Firetly Terminal and SUSHI Version. | ||
| > **_Firely Terminal Pipeline_** 0.4.0. | ||
| > **_SUSHI Versions_** 3.5.0. | ||
| - [ ] By running the _Release_Publish.py_ script, release version and date was updated accordingly. The script ran without errors. | ||
| - [ ] Eventually, increase the dependency of to newer Basis Modul (package and sushi-config) | ||
| - [ ] New Release Notes were created, aglined to the committ history and cleaned. In Github, go to | ||
| - [ ] _-> Releases_ then _-> Draft a new release_ with the _Modul Name and Version_, then | ||
| - [ ] _-> Target the main-Branch_ and _->enter a new Tag according to the Version_, then click. | ||
| - [ ] Click _-> Generate Release notes_ , _->Adjust them if necessary_ and _-> Copy/Paste the Details in the RealeaseNotes.md_ of the very Branch you want to merge. | ||
| - [ ] Finally _-> Save as Draft_ | ||
|
|
||
| ## Merge and Publishing | ||
| <!--- Now lets do the merge. --> | ||
| <!--- Include details of your testing environment, especially --> | ||
| - [ ] With the updated Version, Dates, and Release Notes (as described above) with the last committ into the Branch you want to merge. | ||
| - [ ] In GitHub _-> Actions_ the _->CI (FHIR Validation)_ workflow terminates successfully. | ||
| - [ ] Add the Approve / the PR gets posivitly reviewed by a collegue. | ||
| - [ ] Merge (without squash) the PR, delete the Branch. | ||
|
|
||
|
|
||
| ## Post-Merge Activities | ||
| <!--- Do some update, refreshe and cleanup of related resources. --> | ||
|
|
||
| - [ ] Go to the corresponding SIMPLIFIER Project and _-> Github -> Reimport_ the project. | ||
| - [ ] Go to the corresponding SIMPLIFIER Project and _-> Packages -> Expand the Dropdown for Create -> Create new package_ for the project. | ||
| - [ ] With the corresponding version number, and | ||
| - [ ] The Release notes (from above) and a compare-link to the previous Release. | ||
| - [ ] Unlist the old package by _-> clicking on the old package_, _-> go to Admininstration_ and _-> click on Unlist_ | ||
| - [ ] Publish the previosuly drafteted Release, including version number, on GitHub. | ||
| - [ ] Provide / Archive the IG in the corresponding _gh-pages_ branch of the GitHub project. | ||
| - [ ] Checkout the Branch (no need to merge it later). | ||
| - [ ] Export from Simplifier via _-> Guides -> Expand the Modul ... -> Export_ | ||
| - [ ] Unpack the zip, remove the packages folder (because its kinda big), and move everything else to a (version coressponding) new folder in the branch folder structure. | ||
| - [ ] committ the branch. | ||
|
|
||
|
|
||
| ## Finished |