Skip to content

fix: docs - gas_budget.py path prefix + real CI badge (#122 #123)#126

Open
gtx20060124-bot wants to merge 1 commit into
kcolbchain:mainfrom
gtx20060124-bot:fix-123-readme-paths
Open

fix: docs - gas_budget.py path prefix + real CI badge (#122 #123)#126
gtx20060124-bot wants to merge 1 commit into
kcolbchain:mainfrom
gtx20060124-bot:fix-123-readme-paths

Conversation

@gtx20060124-bot

Copy link
Copy Markdown

Summary

Addresses #123 and #122:

  1. docs (docs: fix missing switchboard/ path prefix for gas_budget.py in README table #123): Fixed missing switchboard/ path prefix for gas_budget.py in the What's in the box table. Was listed as gas_budget.py but should be switchboard/gas_budget.py to match the project structure.

  2. docs (docs(readme): swap hard-coded tests-passing shield for real Actions badge #122): Replaced the static tests-passing badge (which was always green regardless of CI status) with the real GitHub Actions badge pointing to .github/workflows/ci.yml.

Changes

  • README.md: Fixed gas_budget.py path in the module table
  • README.md: Swapped hard-coded badge for real Actions badge

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

@abhicris

Copy link
Copy Markdown
Contributor

Welcome to kcolbchain, @gtx20060124-bot — glad you're here. 🌱

Here's what happens from this PR:

  1. Our automated review looks for obvious issues (tests, secrets, size) within a couple of hours.
  2. If it's clean and CI passes, we merge without back-and-forth.
  3. If we need changes, we'll leave a specific comment — not a generic nit. Push another commit and we re-review.

While you wait:

  • Run the repo's tests locally (see the repo README.md).
  • Keep the PR scoped to one concern — bigger PRs land slower.
  • Don't commit tokens or .env contents.

What happens after your first merge

Thanks for writing the code. We're building this to last.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants