Skip to content

Commit f7b2e54

Browse files
docs: improve documentation for readme and development
Signed-off-by: Nicklas Silversved <nicklas.silversved@digg.se>
1 parent b36ec3b commit f7b2e54

File tree

2 files changed

+64
-27
lines changed

2 files changed

+64
-27
lines changed

README.md

Lines changed: 32 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ npm install
5151
Använd det här kommandot för att köra applikationen mot en YAML-fil:
5252

5353
```bash
54-
npm start -- -f PATH_TO_THE_YAML_FILE
54+
npm start -- -f <PATH>/<YAML_FILE>
5555
```
5656

5757
**Exempel**
@@ -67,7 +67,7 @@ Main och pre-releaser används med reservation för att all funktionalitet inte
6767

6868
## Användning
6969

70-
För att validera mot en specifik kategori av regler, lägg till `-c CategoryName`.
70+
För att validera mot en specifik kategori av regler, lägg till `-c <CategoryName>`.
7171

7272
### Validering med utvalda regler
7373
**Exempel**
@@ -89,7 +89,7 @@ npm start -- -f apis/dok-api.yaml -c DokRules
8989
- UfnRules
9090
- VerRules
9191

92-
För att spara meddelanden från felloggar, lägg till `-l FileName`.
92+
För att spara meddelanden från felloggar, lägg till `-l <FILE>`.
9393

9494

9595
### Validering som skriver resultat till en valfri logfil
@@ -109,7 +109,7 @@ npm start -- -f apis/dok-api.yaml -l rap-lp.log -a
109109
```
110110

111111
### Validering som sparar loggdiagnostik i en fil
112-
För att spara loggdiagnostik i en fil, lägg till `-d FileName`
112+
För att spara loggdiagnostik i en fil, lägg till `-d <FILE>`
113113

114114
**Exempel**
115115

@@ -134,7 +134,7 @@ npm start -- -f apis/dok-api.yaml --dex
134134
**Exempel med sökväg till avstämningsfil i Excel**
135135

136136
```bash
137-
npm start -- -f apis/dok-api.yaml --dex path-to-excel-file
137+
npm start -- -f apis/dok-api.yaml --dex <PATH>
138138
```
139139

140140
### Visa information version
@@ -152,8 +152,32 @@ npm start -- --version
152152
npm start -- --help
153153
```
154154

155+
### Testa mot publicerat npm paket
156+
> OBS se till så att .npmrc verkligen är konfigurerat mot @diggsweden:registry=https://npm.pkg.github.com
157+
158+
För att köra mot ett **publicerat** paket istället för lokala byggen, se [npm paket](https://github.com/diggsweden/rest-api-profil-lint-processor/pkgs/npm/rest-api-profil-lint-processor) för version och använd:
159+
160+
161+
```bash
162+
npm login --registry=https://npm.pkg.github.com
163+
# username: ditt GitHub-användarnamn
164+
# password: GitHub PAT med read:packages
165+
```
166+
167+
> Om du saknar inloggning med GitHub PAT, se [Access till registry](#access-till-registry)
168+
169+
Kör sedan:
170+
```bash
171+
# npx @diggsweden/rest-api-profil-lint-processor@1.0.0 -f openapi-test.yaml -l rap-lp.log -d avstamning.xlsx
172+
npx @diggsweden/rest-api-profil-lint-processor@1.0.0 -f <PATH>/<YAML_FILE>
173+
```
174+
175+
---
176+
155177
### Användning via podman/docker
156178

179+
> För alla kommandon bör ***podman*** kunna ersättas med ***docker*** om så önskas.
180+
157181
I en terminal kör:
158182
```bash
159183
podman run --rm -it -v $(pwd):/<PATH> ghcr.io/diggsweden/rest-api-profil-lint-processor:<VERSION X.X.X> -f <PATH>/<YAML_FILE>
@@ -164,7 +188,7 @@ I en terminal kör:
164188

165189
Exempel
166190
```bash
167-
podman run -it -v $(pwd):/app/example ghcr.io/diggsweden/rest-api-profil-lint-processor:1.0.0 -f example/dot-api.yaml -l example/test.log --dex example/avstamning.xlsx
191+
podman run -it -v $(pwd):/app/example ghcr.io/diggsweden/rest-api-profil-lint-processor:1.0.0 -f example/dot-api.yaml -l example/rap-lp.log --dex example/avstamning.xlsx
168192
```
169193

170194
Vid eventuella fel och du inte hittar rap-lp-error.log kan du behöva köra kommandot via containern enligt den alternativa instruktionen nedan. Se till att containern har rättigheter att skriva till den katalog som du mountar, se [Skrivåtkomst till mount från container](#skrivåtkomst-till-mount-från-container).
@@ -180,7 +204,7 @@ Exempel:
180204
```bash
181205
$ podman run --rm -it --entrypoint /bin/sh -v $(pwd):/apis ghcr.io/diggsweden/rest-api-profil-lint-processor:0.3.0
182206

183-
/app: $ npm start -- -f apis/dot-api.yaml -l test.log --dex example.xlsx
207+
/app: $ npm start -- -f apis/dot-api.yaml -l rap-lp.log --dex example.xlsx
184208
```
185209

186210
### Eventuella hinder med podman-kommando
@@ -209,7 +233,7 @@ Du kan behöva ett Personal Access Token (PAT) för din användare i github för
209233
3. Beroende på din miljö och vilka möjligheter du har, hantera åtkomstproblemet mer beständigt och återställ tidigare läs- och skrivrättigheter.
210234
4. Du kan även prova:
211235
```bash
212-
sudo podman run -it -v $(pwd):/app/example ghcr.io/diggsweden/rest-api-profil-lint-processor:1.0.0 -f example/dot-api.yaml -l example/test.log --dex example/avstamning.xlsx
236+
sudo podman run -it -v $(pwd):/app/example ghcr.io/diggsweden/rest-api-profil-lint-processor:1.0.0 -f example/dot-api.yaml -l example/rap-lp.log --dex example/avstamning.xlsx
213237
```
214238

215239
### Riktlinjer och förklaringar

development/DEVELOPMENT.md

Lines changed: 32 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@ Denna guide beskriver de viktigaste grunderna för utveckling i detta projekt.
2424

2525
### Pull Request-flöde
2626

27-
När du skickar in en PR kommer CI automatiskt att köra flera kontroller. För att undvika överraskningar bör du köra dessa kontroller lokalt först.
27+
När du skickar in en PR kommer CI automatiskt att köra flera kontroller.<br/>
28+
För att undvika överraskningar bör du köra dessa kontroller lokalt först.
2829

2930
#### Förutsättningar
3031
- [Podman](https://podman.io/)
@@ -58,9 +59,26 @@ Om några kontroller misslyckas i CI-pipelinen:
5859
5. Verifiera att alla kontroller passerar i den uppdaterade PR:en
5960

6061
## Releaseprocess
61-
Det finns ett arbetsflöde för att publicera images och paket antingen som en pre-release-version eller som en stabil release.
62+
Det finns två olika workflows för att göra en ny release, antingen för en feature under utveckling eller för produktion.
6263

63-
### Pre-release
64+
- [Utveckling](../.github/workflows/release-dev-workflow.yml)
65+
- [Produktion](../.github/workflows/release-workflow.yml)
66+
67+
För varje ny release kommer det bland annat att det byggas en docker image och ett npm paket som kan hämtas hem och köras. <br>
68+
Projektets releaser bygger på en gemensam struktur för DiggSweden vilken innefattar säkerhetsrutiner, automatiserade releaser och kvalitetskontroller. <br>
69+
För mer information se [Reusable CI/CD Workflows](https://github.com/diggsweden/reusable-ci?tab=readme-ov-file#reusable-cicd-workflows).
70+
71+
### Release för utveckling
72+
Genom att pusha till en branch med prefix **dev/** eller **feat/** skapas en utvecklingsrelease vilket möjliggör tester av paket och images innan produktion.
73+
74+
> **Utvecklingsreleaser kommer ***INTE*** generera och uppdatera changelog, release notes eller göra en ny release i GitHub.**
75+
76+
### Release för produktion
77+
Vid release till produktion kan man välja om man vill släppa det som en pre-release version eller som en stabil version.
78+
79+
> **Produktionsreleaser kommer automatiskt generera och uppdatera changelog, release notes och göra en ny release i GitHub.**
80+
81+
#### Pre-release
6482
Annotera pre-release-taggen med suffix och en version, använd suffix:
6583
- alpha → tidig testversion, ofta instabil
6684
- beta → mer testad, men fortfarande pre-release
@@ -84,13 +102,11 @@ git tag -s -a v1.0.3-alpha.1 -m "v1.0.3-alpha.1"
84102
git push origin v1.0.3-alpha.1
85103
```
86104

87-
### Stabil release
105+
#### Stabil release
88106
Checka ut main, hämta senaste ändringarna och tagga senaste commit på main.
89107

90108
***Säkerställ att tagg-versionen matchar versionen i package.json och package-lock.json.***
91109

92-
Checkout main, pull the latest changes and tag the latest main commit.
93-
94110
```
95111
git checkout <branch>
96112
git pull
@@ -106,22 +122,19 @@ git push origin v1.0.0
106122

107123
***På grund av projektbegränsningar är pre- och stabila releaser från main endast möjliga för admins!***
108124

109-
### Släpp stabil version på GitHub
110-
1. Öppna [releases](https://github.com/diggsweden/rest-api-profil-lint-processor/releases) på GitHub.
111-
2. "Draft a new release"
112-
3. Välj vilken tagg som ska släppas
113-
4. Lägg till en release-titel – vX.X.X
114-
5. Lägg till en beskrivning eller generera release notes och rensa bort onödiga commits från beskrivningen
115-
6. Markera som senaste release
116-
7. Publicera releasen
117-
118125
### Testa och verifiera release
119126
Efter varje release, pre- eller stabil, bör funktionaliteten testas och verifieras.
120127

121-
Detta kan göras via stickprov av följande funktioner:
122-
123128
#### Test av image
124129
Kör följande kommandon med den version av verktyget du vill testa – [använd podman/docker](../README.md#användning-via-podmandocker)
130+
```bash
131+
# EXEMPEL: podman run -it -v $(pwd):/app/example ghcr.io/diggsweden/rest-api-profil-lint-processor:1.0.0 -f example/dot-api.yaml -l example/rap-lp.log --dex example/avstamning.xlsx
132+
podman run --rm -it -v $(pwd):/<PATH> ghcr.io/diggsweden/rest-api-profil-lint-processor:<VERSION X.X.X> -f <PATH>/<YAML_FILE>
133+
```
134+
#### Test av npm-paket
135+
Kör följande kommandon med den version av verktyget du vill testa – [använd node](../README.md#användning)
136+
```bash
137+
# EXEMPEL: npx @diggsweden/rest-api-profil-lint-processor@1.0.0 -f openapi-test.yaml -l rap-lp.log -d avstamning.xlsx
138+
npx @diggsweden/rest-api-profil-lint-processor@1.0.0 -f <PATH>/<YAML_FILE>
139+
```
125140

126-
### Test av npm-paket
127-
...

0 commit comments

Comments
 (0)