Repository navigation
ci: add build check - #16
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Dependabot autopilot merges a security update only when a real build or test check passes on the PR. This repo had no CI, so no update could qualify, and people's PRs had no build check either.
What the job runs
One job,
build:uv venvanduv pip install -r frameworks/tinker/requirements.txt(the pinned file Dependabot watches).python -m unittest discover -s testsandpython -m unittest discover -s frameworks/tinker -p 'test_*.py'.The tests use
unittest, as the README documents, so the job runs them withunittestinstead ofpytest.The job needs no secrets. It installs from public registries and does not change any lockfile or dependency. Actions are pinned by commit SHA. It runs on every PR and on pushes to the default branch, with read-only
contentspermission and a 20 minute timeout.Note
Low Risk
Adds CI configuration only; no application, auth, or dependency lockfile changes.
Overview
Adds a GitHub Actions CI workflow so every pull request and push to
mastergets an automated build gate—enabling Dependabot to require a passing check before merging security updates.The single
buildjob runs on Ubuntu with read-only repo permissions, SHA-pinned checkout/setup actions, and concurrency that cancels duplicate runs. It creates a Python 3.12 venv with uv, installsframeworks/tinker/requirements.txt(the pinned file Dependabot tracks), then runs the documented offlineunittestsuites undertestsandframeworks/tinker.Reviewed by Cursor Bugbot for commit 16e3e3d. Bugbot is set up for automated code reviews on this repo. Configure here.