Skip to content

CI: TripDeck's shared Bun workflow, package cache, dependency review - #18

Merged
Torgh merged 1 commit into
mainfrom
ci-quick-wins
Oct 25, 2026
Merged

Torgh merged 1 commit into
mainfrom
ci-quick-wins

Conversation

@Torgh

@Torgh Torgh commented Oct 25, 2026

Copy link
Copy Markdown
Owner

What

  • ci.yml calls Torgh/tripdeck/.github/workflows/bun-app.yml@main (CI: shared Bun workflow, package cache, dependency review tripdeck#44), so both apps run the same checks. It follows TripDeck's main rather than a pinned commit: a pin would stay old, since Dependabot only moves pins that follow version tags.
  • Bun package cache and dependency review on pull requests come with it.
  • Repo settings (already done): Dependabot alerts and security updates turned on – they were off here, unlike TripDeck.
  • README: the CI section.

Before merging

  1. Merge CI: shared Bun workflow, package cache, dependency review tripdeck#44 first, then re-run this PR's CI (until then bun-app.yml@main doesn't exist).
  2. The check names change to App / Typecheck, lint, tests with coverage, build and App / Tests on Windows (+ App / Dependency review), so the required checks on main need updating.

🤖 Generated with Claude Code

ci.yml now calls bun-app.yml from Torgh/tripdeck@main, so both apps run
the same checks: typecheck, lint, tests with coverage and build on Linux,
tests on Windows, with Bun's package cache kept between runs, and on pull
requests a dependency review that fails on a new dependency with a known
high or critical vulnerability. Dependabot alerts and security updates are
now on for this repository too.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Torgh Torgh closed this Oct 25, 2026
@Torgh Torgh reopened this Oct 25, 2026
@Torgh
Torgh merged commit 58feeda into main Oct 25, 2026
8 checks passed
@Torgh
Torgh deleted the ci-quick-wins branch October 25, 2026 16:16
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.

1 participant