Skip to content

Commit 5de65f7

Browse files
nolteclaude
andcommitted
chore(docs): proofread README and docs prose for grammar and style
Fix broken grammar and inconsistent capitalisation in the README, unify the German address form to the dominant impersonal infinitive, spell out abbreviations on first use, and align em-dash spacing and a heading to the established project style. README is Vale-clean and all include-markdown markers are preserved. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent a9124ac commit 5de65f7

6 files changed

Lines changed: 35 additions & 35 deletions

File tree

README.md

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ Centralised reusable [GitHub Actions workflows](https://docs.github.com/en/actio
1616
| ```reusable-ansible-galaxy-push.yaml``` | Publish an Ansible role to [Ansible Galaxy](https://galaxy.ansible.com/) via [robertdebock/galaxy-action](https://github.com/robertdebock/galaxy-action). |
1717
| ```reusable-ansible-molecule.yaml``` | Run a [Molecule](https://molecule.readthedocs.io/) test scenario via [gofrolist/molecule-action](https://github.com/gofrolist/molecule-action). |
1818
| ```reusable-automerge.yaml``` | Auto-merge pull requests via [pascalgn/automerge-action](https://github.com/pascalgn/automerge-action). |
19-
| ```reusable-chain-bench.yaml``` | Run the supply-chain CIS benchmark via [aquasecurity/chain-bench-action](https://github.com/aquasecurity/chain-bench-action). |
19+
| ```reusable-chain-bench.yaml``` | Run the supply-chain CIS (Center for Internet Security) benchmark via [aquasecurity/chain-bench-action](https://github.com/aquasecurity/chain-bench-action). |
2020
| ```reusable-dependency-review.yaml``` | Gate pull requests against vulnerable or licence-incompatible dependency changes via [actions/dependency-review-action](https://github.com/actions/dependency-review-action). |
2121
| ```reusable-docker-lint-build.yaml``` | Run [hadolint](https://github.com/hadolint/hadolint-action) on a Dockerfile and a buildx dry-build (no push) for fast PR feedback. |
2222
| ```reusable-docker-publish.yaml``` | Build a multi-arch container image with [docker/build-push-action](https://github.com/docker/build-push-action) and push it to a configurable OCI registry. |
@@ -28,29 +28,29 @@ Centralised reusable [GitHub Actions workflows](https://docs.github.com/en/actio
2828
| ```reusable-spelling-vale.yaml``` | Lint Markdown prose via [errata-ai/vale-action](https://github.com/errata-ai/vale-action) with inline reviewdog annotations on pull requests. |
2929
| ```reusable-sphinx.yaml``` | Build and publish a [Sphinx](https://www.sphinx-doc.org/en/master) documentation site to [GitHub Pages](https://pages.github.com/). |
3030
| ```reusable-stale.yaml``` | Mark and close stale issues and pull requests via [actions/stale](https://github.com/actions/stale). |
31-
| ```reusable-trivy.yaml``` | Scan the GitRepo by using [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action). |
32-
| ```reusable-tf-lint.yaml``` | Use [terraform-linters/setup-tflint](https://github.com/terraform-linters/setup-tflint) for Lint terraform sources. |
31+
| ```reusable-trivy.yaml``` | Scan the repository with [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action). |
32+
| ```reusable-tf-lint.yaml``` | Lint Terraform sources with [terraform-linters/setup-tflint](https://github.com/terraform-linters/setup-tflint). |
3333

3434
<!--td-workflows-end-->
3535

3636

3737
## Probot configuration
3838

3939
<!--probot-intro-start-->
40-
Collection of common Configurations for Project Management and CI/CD.
41-
For Using in other GitHub Projects, having a reusable set of Probot Configuration Repository, more information at [probot.github.io](https://probot.github.io/docs/best-practices/#configuration).
40+
Collection of common configurations for project management and CI/CD.
41+
Reuse this shared set of Probot configurations across your GitHub projects. For more information, see the [Probot best-practices documentation](https://probot.github.io/docs/best-practices/#configuration).
4242
<!--probot-intro-end-->
4343

4444
<!--td-probot-apps-start-->
4545
| probot | git | description |
4646
|-------------------------------------------------------------------|-----------------------------------------------------------------------------|---------------------------------------------------------------------------------------------------------------------|
47-
| [boring-cyborg](https://probot.github.io/apps/boring-cyborg/) | [kaxil/boring-cyborg](https://github.com/kaxil/boring-cyborg) | Different actions like, automatically label Pull Request |
47+
| [boring-cyborg](https://probot.github.io/apps/boring-cyborg/) | [kaxil/boring-cyborg](https://github.com/kaxil/boring-cyborg) | Automates pull-request labelling and other repository housekeeping tasks. |
4848
| [release-drafter](https://probot.github.io/apps/release-drafter/) | [release-drafter/release-drafter](https://github.com/release-drafter/release-drafter) | Creates a human-readable release changelog (**deprecated** as a Probot app—prefer the workflow implementation). |
49-
| [renovate](https://github.com/apps/renovate) | | Using [renovate](https://www.whitesourcesoftware.com/free-developer-tools/renovate/) for keep dependencies in sync. |
49+
| [renovate](https://github.com/apps/renovate) | | Keeps dependencies in sync with [Renovate](https://www.whitesourcesoftware.com/free-developer-tools/renovate/), an automated dependency-update bot. |
5050
| [settings](https://probot.github.io/apps/settings/) | [probot/settings](https://github.com/probot/settings) | Configure GitHub Projects by Source. |
5151
<!--td-probot-apps-end-->
5252

53-
For More information take a look to the GH Page, [gh-plumbing](http://nolte.github.io/gh-plumbing).
53+
For more information, see the [gh-plumbing GitHub Pages site](http://nolte.github.io/gh-plumbing).
5454

5555
## Development
5656

@@ -63,13 +63,13 @@ asdf install
6363

6464
### Workflows
6565

66-
For local testing you can use [nektos/act](https://github.com/nektos/act), run the GitHub Actions locally.
66+
For local testing, use [nektos/act](https://github.com/nektos/act) to run the GitHub Actions on your machine.
6767

6868
```sh
6969
act push -j static -W .github/workflows/build-static-tests.yaml
7070
```
7171

72-
Will be start the [![.github/workflows/build-static-tests.yaml](https://github.com/nolte/gh-plumbing/actions/workflows/build-static-tests.yaml/badge.svg)](https://github.com/nolte/gh-plumbing/actions/workflows/build-static-tests.yaml) at your system.
72+
This runs the [`build-static-tests`](https://github.com/nolte/gh-plumbing/actions/workflows/build-static-tests.yaml) workflow on your machine.
7373

7474

7575
### Documentation
@@ -83,7 +83,7 @@ pip install -r requirements-dev.txt
8383
mkdocs serve -a localhost:8001
8484
```
8585

86-
Open [localhost:8001](http://localhost:8001/) for take a look to the latest documentation, created with [mkdocs](https://www.mkdocs.org/).
86+
Open [localhost:8001](http://localhost:8001/) to view the latest documentation, built with [mkdocs](https://www.mkdocs.org/).
8787
<!--development-intro-end-->
8888

8989
### Task

docs/de/development/index.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ act push -j static -W .github/workflows/build-static-tests.yaml
5959

6060
---
6161

62-
## Prose-Linting
62+
## Prosa-Linting
6363

6464
[Vale](https://vale.sh/) prüft Markdown-Dateien im CI über `reusable-spelling-vale.yaml`. Die Regeln liegen in `.vale.ini`, die Styles unter `.github/styles/`.
6565

docs/de/portfolio-app/setup.md

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -24,18 +24,18 @@ Credential, das GitHub als user-initiated wertet.
2424

2525
Die Spec verlangt eine Portfolio-Level-Lösung: eine App, in jedem
2626
Konsument-Repo installiert, mit demselben Wrapper-Pattern in jedem
27-
`.github/workflows/`. Kein PAT-Sammeln pro Repo, kein
28-
personengebundenes Credential.
27+
`.github/workflows/`. Kein Sammeln von Personal-Access-Tokens (PAT)
28+
pro Repo, kein personengebundenes Credential.
2929

3030
---
3131

3232
## Owner-Modi
3333

3434
Die Portfolio-App kann Konsumenten unter einer GitHub-**Organisation**
35-
oder unter einem persönlichen **User-Account** bedienen. Wähle den
36-
Modus, der zum Account passt, der die Konsumenten-Repositories
37-
besitzt — beide Modi liefern dasselbe nachgelagerte Verhalten
38-
(App-emittierte Events kaskadieren user-initiated), nur die
35+
oder unter einem persönlichen **User-Account** bedienen. Den Modus
36+
wählen, der zum Account passt, der die Konsumenten-Repositories
37+
besitzt. Beide Modi liefern dasselbe nachgelagerte Verhalten
38+
App-emittierte Events kaskadieren user-initiated, nur die
3939
Credential-Verkabelung unterscheidet sich.
4040

4141
| Modus | Wann wählen | Credentials liegen als |
@@ -108,12 +108,12 @@ Jedes aktivierte Häkchen würde nur tote Webhook-Versuche erzeugen.
108108
`https://github.com/settings/apps/new` für einen persönlichen
109109
Account). Namensvorschlag: `nolte-portfolio-bot`.
110110
2. **Permissions wie oben vergeben** und jeden Webhook-Event abwählen.
111-
3. **Private Key (PEM) generieren** einmal speichern; GitHub zeigt
112-
ihn nicht erneut an.
111+
3. **Private Key im PEM-Format (Privacy-Enhanced Mail) generieren**
112+
einmal speichern; GitHub zeigt ihn nicht erneut an.
113113
4. **App-ID notieren** (sichtbar auf der App-Settings-Seite).
114114
5. **App installieren** in jedem Konsument-Repository, das
115-
cascade-korrekte Workflows braucht. Beginne mit
116-
`nolte/gh-plumbing` selbst.
115+
cascade-korrekte Workflows braucht. Mit `nolte/gh-plumbing` selbst
116+
beginnen.
117117
6. **Credentials setzen** im Scope, der zum Owner-Modus passt:
118118
- **Organisations-Modus:** eine
119119
[Org-Level-Actions-Variable](https://docs.github.com/en/actions/learn-github-actions/variables)
@@ -149,10 +149,10 @@ Jedes aktivierte Häkchen würde nur tote Webhook-Versuche erzeugen.
149149
## Wrapper-Pattern (für nachgelagerte Konsumenten)
150150

151151
Die Cascade-emittierenden Wrapper in `nolte/gh-plumbing` zeigen das
152-
Pattern. Übernimm dieselbe Form in dein Repository für jeden Wrapper,
153-
der eine `reusable-*.yaml` aufruft, deren Arbeit nachgelagerte
154-
Workflows triggern soll oder die du unter derselben Release-Audit-
155-
Identität halten willst.
152+
Pattern. Dieselbe Form in das eigene Repository für jeden Wrapper
153+
übernehmen, der eine `reusable-*.yaml` aufruft, deren Arbeit
154+
nachgelagerte Workflows triggern soll oder die unter derselben
155+
Release-Audit-Identität bleiben soll.
156156

157157
```yaml title=".github/workflows/automerge.yaml"
158158
on:
@@ -233,7 +233,7 @@ Eine dritte Self-Check-Zeile betrifft den Issue-Lifecycle:
233233
|---|---|
234234
| PR squash-mergen, dessen Body `Closes #N` enthält | Issue `#N` flippt automatisch auf `CLOSED`; `gh issue view N --json state` liefert `CLOSED`. Setzt die App-Permission `Issues: Read and write` voraus — ohne sie greift der Merge zwar, der Close feuert aber nicht, obwohl `gh pr view --json closingIssuesReferences` den Autolink korrekt geparst zeigt. |
235235

236-
Wenn ein Cascade immer noch nicht triggert, prüfe:
236+
Wenn ein Cascade immer noch nicht triggert, prüfen:
237237

238238
1. Die App ist im Konsument-Repo installiert (`Settings → GitHub Apps`).
239239
2. `vars.PORTFOLIO_APP_ID` ist für den Workflow sichtbar.
@@ -254,7 +254,7 @@ Wenn ein Cascade immer noch nicht triggert, prüfe:
254254
| Reguläre jährliche Rotation | Neuen Private Key auf der App-Seite generieren, `PORTFOLIO_APP_PRIVATE_KEY` in jedem Konsumenten aktualisieren (oder einmal auf Org-Level, falls org-scoped), den alten Key auf der App-Seite löschen. |
255255
| Vermuteter Key-Leak | Geleakten Key sofort in den App-Settings widerrufen, neuen Key generieren, am selben Tag verteilen. App-ID bleibt unverändert. |
256256
| App-ID-Rotation | Niemals nötig — App-IDs sind unveränderlich. |
257-
| Permission-Scope-Änderung | App-Permissions editieren; bestehende Installation-Tokens mit alten Permissions laufen für ihre TTL (≤ 1 h) weiter, dann werden sie mit neuem Scope erneuert. |
257+
| Permission-Scope-Änderung | App-Permissions editieren; bestehende Installation-Tokens mit alten Permissions laufen für ihre Gültigkeitsdauer (TTL, ≤ 1 h) weiter, dann werden sie mit neuem Scope erneuert. |
258258

259259
---
260260

docs/de/probot/settings.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ repository:
2626
```
2727
2828
!!! tip "Overrides"
29-
Schlüssel in deiner lokalen `.github/settings.yml` überschreiben die geerbten Werte. Definiere dort nur, was vom geteilten Default abweicht.
29+
Schlüssel in der lokalen `.github/settings.yml` überschreiben die geerbten Werte. Dort nur angeben, was vom geteilten Default abweicht.
3030

3131
---
3232

docs/de/workflows/release.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,11 +66,11 @@ jobs:
6666
!!! note "Tag muss zu einem bestehenden Entwurf passen"
6767
`tag` muss zum Tag eines bestehenden Release-Drafter-Entwurfs passen. Es
6868
gibt keine „neuester gewinnt"-Heuristik — existiert kein Entwurf für den
69-
angegebenen Tag, scheitert der Workflow sofort. Lass vorher den
70-
Draft-Workflow auf `develop` laufen.
69+
angegebenen Tag, scheitert der Workflow sofort. Zuvor den
70+
Draft-Workflow auf `develop` laufen lassen.
7171

7272
!!! tip "Dry Run"
73-
Setze `dry_run: true`, um jeden Validierungs-Gate auszuführen, ohne den
73+
`dry_run: true` setzen, um jeden Validierungs-Gate auszuführen, ohne den
7474
Entwurf zu einem veröffentlichten Release umzuschalten. Nützlich, um den
7575
Publish-Pfad vor dem eigentlichen Release zu verifizieren.
7676

@@ -91,7 +91,7 @@ jobs:
9191
```
9292

9393
!!! warning "Direkte Commits auf master"
94-
Commit nicht direkt auf `master` — der Workflow überschreibt deine Änderungen beim nächsten Release.
94+
Nicht direkt auf `master` committen — der Workflow überschreibt lokale Änderungen beim nächsten Release.
9595

9696
---
9797

docs/en/portfolio-app/setup.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ permissions**:
5858
|---|---|---|
5959
| `Contents` | `Read and write` | Squash-merge to develop, edit releases, fast-forward master |
6060
| `Pull requests` | `Read and write` | `pascalgn/automerge-action` reads and merges PRs |
61-
| `Issues` | `Read and write` | Auto-close issues referenced via `Closes #N` / `Fixes #N` / `Resolves #N` in the PR body when the App squash-merges. GitHub only honours these autolinks when the merging actor has `Issues: write`; without this scope the autolinks parse correctly into `closingIssuesReferences` but the close never fires (observed end-to-end on #357 / #358). |
61+
| `Issues` | `Read and write` | Automatically close issues referenced via `Closes #N` / `Fixes #N` / `Resolves #N` in the PR body when the App squash-merges. GitHub only honours these autolinks when the merging actor has `Issues: write`; without this scope the autolinks parse correctly into `closingIssuesReferences` but the close never fires (observed end-to-end on #357 / #358). |
6262
| `Actions` | `Read-only` | `release-publish` reads `gh run list` for the post-publish cascade sanity check |
6363
| `Metadata` | `Read-only` | Mandatory baseline (GitHub sets this automatically and won't let you disable it) |
6464

@@ -226,7 +226,7 @@ A third self-check covers the issue lifecycle:
226226

227227
| Action | Expected outcome |
228228
|---|---|
229-
| Squash-merge a PR whose body says `Closes #N` | Issue `#N` flips to `CLOSED` automatically; `gh issue view N --json state` returns `CLOSED`. Requires the App's `Issues: Read and write` permissionwithout it the merge succeeds but the close never fires, even though `gh pr view --json closingIssuesReferences` shows the link was parsed. |
229+
| Squash-merge a PR whose body says `Closes #N` | Issue `#N` flips to `CLOSED` automatically; `gh issue view N --json state` returns `CLOSED`. Requires the App's `Issues: Read and write` permissionwithout it the merge succeeds but the close never fires, even though `gh pr view --json closingIssuesReferences` shows that GitHub parsed the link. |
230230

231231
If a cascade still fails to fire, check:
232232

0 commit comments

Comments
 (0)