Skip to content

Add real CI - #19

Merged
pbrisbin merged 2 commits into
mainfrom
pb/ci
Jul 30, 2026
Merged

Add real CI#19
pbrisbin merged 2 commits into
mainfrom
pb/ci

Conversation

@pbrisbin

Copy link
Copy Markdown
Member

Our compliance automation added a CI workflow at some point. It
incorrectly uses a custom runner and tries to run JavaScript tests.

I was first going to just fix the runner, because it's holding up
Renovate PRs, but then I noticed the tests too so I replaced it by
running install (and asserting success, implicitly).

pbrisbin added 2 commits July 30, 2026 10:18
Our compliance automation added a CI workflow at some point. It
incorrectly uses a custom runner and tries to run JavaScript tests.

I was first going to just fix the runner, because it's holding up
Renovate PRs, but then I noticed the tests too so I replaced it by
running `install` (and asserting success, implicitly).
@pbrisbin
pbrisbin requested a review from danroyo July 30, 2026 15:15
@pbrisbin
pbrisbin marked this pull request as ready for review July 30, 2026 15:15
@pbrisbin
pbrisbin requested a review from a team as a code owner July 30, 2026 15:15
@pbrisbin
pbrisbin merged commit be81c0d into main Jul 30, 2026
2 checks passed
@pbrisbin
pbrisbin deleted the pb/ci branch July 30, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants