Skip to content

Remove outdated information #5218

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.

Sign up for GitHub

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

Open
Vince-janv wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Remove outdated information #5218

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0636a01
Remove outdated information
Vince-janv Jul 24, 2026
f94ccb2
Merge branch 'master' into 5217-clean-up-documentation
Vince-janv Jul 24, 2026
5acbdc8
Re-write deployment description
Vince-janv Jul 24, 2026
434aa96
Merge remote-tracking branch 'origin/5217-clean-up-documentation' int…
Vince-janv Jul 24, 2026
720a461
Fix incorrect links in DEPLOYMENT.md
Vince-janv Jul 24, 2026
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
1 change: 0 additions & 1 deletion .dockerignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,4 @@ cg.egg-info
.gitlint.yaml
.pre-commit-config.yaml
DEPLOYMENT.md
CONTRIBUTING.md
Dockerfile
38 changes: 0 additions & 38 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop

@Vince-janv Vince-janv Jul 24, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@diitaz93 What do you think of this change? Do you thing the other headings except Description and added/changed/fixed are useful?

Original file line number Diff line number Diff line change
Expand Up @@ -11,41 +11,3 @@
### Fixed

-


### How to prepare for test

- [ ] Ssh to relevant server (depending on type of change)
- [ ] Use stage: `us`
- [ ] Paxa the environment: `paxa`
- [ ] Install on stage (example for Hasta):
```shell
bash /home/proj/production/servers/resources/hasta.scilifelab.se/update-tool-stage.sh -e S_cg -t cg -b [THIS-BRANCH-NAME] -a
```
Comment on lines -22 to -24

@diitaz93 diitaz93 Aug 4, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I use this shortcut very often; it would be great to keep it somehow. Maybe the rest of the items in this section can go.


### How to test

- [ ] Do ...

### Expected test outcome

- [ ] Check that ...
- [ ] Take a screenshot and attach or copy/paste the output.
Comment on lines -26 to -33

@diitaz93 diitaz93 Aug 4, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok with removing this. Maybe we could add a section reminding us to paste test results in the comments, hiding sensitive information, but if you find it superfluous, let's just remove it


## Review

- [ ] Tests executed by
- [ ] "Merge and deploy" approved by
Thanks for filling in who performed the code review and the test!

### This [version](https://semver.org/) is a

- [ ] **MAJOR** - when you make incompatible API changes
- [ ] **MINOR** - when you add functionality in a backwards compatible manner
- [ ] **PATCH** - when you make backwards compatible bug fixes or documentation/instructions

## Implementation Plan

- [ ] Document in ...
- [ ] Deploy this branch on ...
- [ ] Inform to ...
Loading
Loading