Skip to content

Commit

Permalink
feat: refactor woke for SSoT
Browse files Browse the repository at this point in the history
- Replace woke command in GitHub action with call to existing make woke
- Makefile needs sudo to install Snap if woke missing
- Missing-woke problem solved by sudo snap install; no need for error
  exit
  • Loading branch information
lblythen committed Aug 13, 2023
1 parent 4684251 commit d030035
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 8 deletions.
16 changes: 9 additions & 7 deletions .github/workflows/automatic-doc-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,15 @@ jobs:
- name: Checkout
uses: actions/checkout@v3

- name: woke
uses: get-woke/woke-action@v0
with:
# Cause the check to fail on any broke rules
fail-on-error: true
workdir: .
woke-args: "*.rst **/*.rst -c https://github.com/canonical-web-and-design/Inclusive-naming/raw/main/config.yml"
- name: Install the doc framework
working-directory: .
run: |
make install
- name: Run incluse-language checker
working-directory: .
run: |
make woke
linkcheck:
name: Link check
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ linkcheck:
. $(VENV) ; $(SPHINXBUILD) -c . -b linkcheck "$(SOURCEDIR)" "$(BUILDDIR)"

woke:
type woke >/dev/null 2>&1 || { snap install woke; exit 1; }
type woke >/dev/null 2>&1 || { sudo snap install woke; }
woke *.rst **/*.rst -c https://github.com/canonical-web-and-design/Inclusive-naming/raw/main/config.yml

.PHONY: help Makefile
Expand Down

0 comments on commit d030035

Please sign in to comment.