Skip to content

Commit

Permalink
Update CI to fail if REAMDE.md was updated manually (#270)
Browse files Browse the repository at this point in the history
  • Loading branch information
bakhtin committed Jul 6, 2023
1 parent d48c2c0 commit 0bc5a88
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 5 deletions.
12 changes: 10 additions & 2 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,19 @@ jobs:
- name: Set up Helm
uses: azure/[email protected]
with:
version: v3.8.1
version: v3.12.1

- uses: actions/setup-python@v4
with:
python-version: "3.10"
python-version: "3.11"

- name: Install pre-commit and helm-docs
run: |
pip install pre-commit==3.3.3
curl -fSsL -o - https://github.com/norwoodj/helm-docs/releases/download/v1.11.0/helm-docs_1.11.0_Linux_x86_64.tar.gz | tar -xz -C /usr/local/bin helm-docs
- name: Run pre-commit hooks
run: pre-commit run --all-files

- name: Set up chart-testing
uses: helm/[email protected]
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
- name: Install Helm
uses: azure/[email protected]
with:
version: v3.8.1
version: v3.12.1

- name: Install additional Helm repos
run: |
Expand Down
7 changes: 6 additions & 1 deletion charts/node/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,12 @@ DO NOT EDIT README.md manually!
We're using [helm-docs](https://github.com/norwoodj/helm-docs) to render values of the chart.
If you updated values.yaml file make sure to render a new README.md locally before submitting a Pull Request.
If you're using [pre-commit](https://pre-commit.com/) REAMDE.md will be updated automatically.
If you're using [pre-commit](https://pre-commit.com/) make sure to install the hooks first:
```
pre-commit install
```
REAMDE.md will be updating automatically after that.
Otherwise, you should install helm-docs and manually update README.md. Navigate to repository root and run:
`helm-docs --chart-search-root=charts/node --template-files=README.md.gotmpl`
Expand Down
7 changes: 6 additions & 1 deletion charts/node/README.md.gotmpl
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,12 @@ DO NOT EDIT README.md manually!
We're using [helm-docs](https://github.com/norwoodj/helm-docs) to render values of the chart.
If you updated values.yaml file make sure to render a new README.md locally before submitting a Pull Request.

If you're using [pre-commit](https://pre-commit.com/) REAMDE.md will be updated automatically.
If you're using [pre-commit](https://pre-commit.com/) make sure to install the hooks first:
```
pre-commit install
```
REAMDE.md will be updating automatically after that.

Otherwise, you should install helm-docs and manually update README.md. Navigate to repository root and run:
`helm-docs --chart-search-root=charts/node --template-files=README.md.gotmpl`

Expand Down

0 comments on commit 0bc5a88

Please sign in to comment.