-
Notifications
You must be signed in to change notification settings - Fork 20
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
add release checklist #215
Merged
jsuchal
merged 21 commits into
slovensko-digital:main
from
solver-it-sro:release-checklist
Aug 1, 2023
Merged
Changes from 8 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
4b63b08
add release checklist
pomali cf312a1
imrpove checklist
pomali d558998
auto-create release issue
pomali 10f8841
add smoke tests
pomali 968febc
test release
pomali 5cc79e8
ready to review
pomali 6324dcd
Merge branch 'main' into release-checklist
pomali 5a9cdc1
fix and add tests for cli to checklist
pomali 48e59b7
split package and prerelease
pomali ff2ad6b
test
pomali b71c6ca
test
pomali 6d89dd4
fix
pomali a901fe0
Merge branch 'main' into release-checklist
pomali de7290b
move test specific constructor to test
pomali 97023da
Merge branch 'main' into release-checklist
pomali e175959
test workflow
pomali b1361b3
fix
pomali 2f46bf8
test
pomali 2226cda
cleanup
pomali 4905e6d
Merge branch 'main' into release-checklist
pomali e5d0f05
Merge branch 'main' into release-checklist
pomali File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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 | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,43 @@ | ||||||
--- | ||||||
name: Release checklist | ||||||
about: Použi tento checklist na testovanie artefaktov pred public releaseom | ||||||
title: "Release " | ||||||
labels: release | ||||||
assignees: @slovensko-digital/autogram-release-team | ||||||
--- | ||||||
|
||||||
## Windows | ||||||
|
||||||
- [ ] funguje inštalácia na Windows cez stiahnutý .msi | ||||||
- [ ] funguje spustenie v GUI móde | ||||||
- [ ] funguje URL handler [autogram://go](autogram://go) | ||||||
- [ ] funguje GUI otvoriť jeden súbor, ten sa zobrazí, viem ho podpísať, vytvorí sa podpísaný súbor | ||||||
- [ ] funguje CLI `autogram --help` | ||||||
|
||||||
## Linux | ||||||
- [ ] funguje inštalácia na Linux (Debian-based) cez stiahnutý .deb | ||||||
- [ ] funguje inštalácia na Linux cez stiahnutý .rpm | ||||||
- [ ] funguje spustenie v GUI móde | ||||||
- [ ] funguje URL handler [autogram://go](autogram://go) | ||||||
- [ ] funguje GUI otvoriť jeden súbor, ten sa zobrazí, viem ho podpísať, vytvorí sa podpísaný súbor | ||||||
- [ ] funguje CLI `autogram --help` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
## MacOS | ||||||
- [ ] funguje inštalácia na MacOS cez stiahnutý .pkg | ||||||
- [ ] funguje spustenie v GUI móde | ||||||
- [ ] funguje URL handler [autogram://go](autogram://go) | ||||||
- [ ] funguje GUI otvoriť jeden súbor, ten sa zobrazí, viem ho podpísať, vytvorí sa podpísaný súbor | ||||||
- [ ] funguje CLI `/Applications/Autogram.app/Contents/MacOS/AutogramApp --help` | ||||||
|
||||||
|
||||||
## Na aspoň jednom systéme | ||||||
|
||||||
- [ ] fungujú všetky smoke testy `./mvnw test -Psmoke` | ||||||
- [ ] funguje spustenie v GUI serverovom móde `autogram --url=autogram://listen?protocol=http&port=37201` na inom porte | ||||||
- [ ] funguje CLI `autogram --cli --source source.pdf` | ||||||
- [ ] funguje CLI `autogram --cli --source source.pdf --target target.pdf` | ||||||
- [ ] funguje CLI `autogram --cli --source source-dir --target target-dir` | ||||||
- [ ] funguje API info request | ||||||
- [ ] funguje API docs request | ||||||
- [ ] funguje API sign request | ||||||
- [ ] funguje s [extension](https://github.com/slovensko-digital/autogram-extension) |
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
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,33 @@ | ||
Release robí slovensko.digital, ale checklist je verejný aby mohol každý contributor samostatne | ||
pretestovať zmeny pred tým než urobí PR. | ||
|
||
# Release Checklist | ||
|
||
## Kontrola nového kódu | ||
|
||
Je potrebné skontrolovať či nepribudol škodlivý kód, exfiltrácia secrets cez GH actions, či je nová core funkcionalita pokrytá testami. | ||
|
||
## Testing | ||
|
||
- [ ] zelené všetky automatizované testy | ||
- [ ] funguje inštalácia na Windows cez stiahnutý .msi | ||
- [ ] funguje inštalácia na MacOS cez stiahnutý .pkg | ||
- [ ] funguje inštalácia na Linux (Debian-based) cez stiahnutý .deb | ||
- [ ] funguje inštalácia na Linux cez stiahnutý .rpm | ||
- [ ] funguje spustenie v GUI móde | ||
- [ ] funguje spustenie v GUI serverovom móde `autogram --url=autogram://listen?protocol=http&port=37200` | ||
- [ ] funguje URL handler na Windows [autogram://go](autogram://go) | ||
- [ ] funguje URL handler na MacOS [autogram://go](autogram://go) | ||
- [ ] funguje URL handler na Linux [autogram://go](autogram://go) | ||
- [ ] funguje GUI otvoriť jeden súbor, ten sa zobrazí, viem ho podpísať, vytvorí sa podpísaný súbor | ||
- [ ] funguje CLI `autogram --cli --source source.pdf` | ||
- [ ] funguje CLI `autogram --cli --source source.pdf --target target.pdf` | ||
- [ ] funguje CLI `autogram --cli --source source-dir --target target-dir` | ||
- [ ] funguje API info request | ||
- [ ] funguje API docs request | ||
- [ ] funguje API sign request | ||
- [ ] funguje s [extension](https://github.com/slovensko-digital/autogram-extension) | ||
|
||
## Pripraviť popis releasu | ||
|
||
vid. existujúce [Releases](https://github.com/slovensko-digital/autogram/releases) |
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
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
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tuto by som skusal cez /docs viac typov suborov, mozno @celuchmarek vie identifikovat take problematicke. Ja si pamatam, ze problem bol naposledy s tym xslt na velke subory a to by toto neodhalilo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ja nevnimam tento checklist ako uplne pretestovanie, skor take, ze robis hociktory release tak toto chces urcite prejst aj ked je to iba nejaka minor zmena