Skip to content

Commit

Permalink
Allow manual runs of the security check (#233)
Browse files Browse the repository at this point in the history
* Allow manual runs of the security check

So we can check before releasing that the release branch is up-to-date.

* Work around markdown-lint-check issue

Pins the version to the last known version that works with a config file, like we use.
  • Loading branch information
shakuzen authored Dec 4, 2024
1 parent da379fc commit 58e9640
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/security.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ name: security

# We don't scan documentation-only commits.
on: # yamllint disable-line rule:truthy
workflow_dispatch: # trigger ad-hoc runs of this action

Check warning on line 6 in .github/workflows/security.yml

View workflow job for this annotation

GitHub Actions / lint

6:22 [comments] too few spaces before comment
push: # non-tagged pushes to master
branches:
- master
Expand Down
2 changes: 1 addition & 1 deletion RELEASE.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ This repo uses semantic versions. Please keep this in mind when choosing version

Before you start a release, make sure all dependencies are up-to-date, or are documented why not.
Pay special attention to the [security workflow](.github/workflows/security.yml), which should
run clean.
run clean. You can manually run the Action to check the release branch is up-to-date.

1. **Alert others you are releasing**

Expand Down
3 changes: 2 additions & 1 deletion build-bin/configure_lint
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
#!/bin/sh -ue

# Attempt to install markdown-link-check if absent
markdown-link-check -V || npm install -g markdown-link-check
# Pinned until https://github.com/tcort/markdown-link-check/issues/369
markdown-link-check -V || npm install -g [email protected]

# Attempt to install yamllint if absent
yamllint || pip install --user yamllint

0 comments on commit 58e9640

Please sign in to comment.