feat: bounty creation ui formatting - #32
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates contributor-facing workflow and documentation around PR preflight checks by adding a bun run pr convenience script, documenting it, and introducing a GitHub Actions CI workflow to run the same checks on pull requests.
Changes:
- Add
bun run prscript to run format/lint/check/test/build in sequence. - Add CI workflow to run lint/check/test/build on every PR (and on manual dispatch), including Playwright browser install for vitest browser mode.
- Update README and PR template to reflect the new PR preflight workflow.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| README.md | Documents bun run pr and clarifies what CI runs on PRs. |
| package.json | Adds a pr script and tweaks scripts section formatting. |
| .github/workflows/ci.yml | Introduces PR CI pipeline for lint/check/test/build with Bun + Playwright. |
| .github/pull_request_template.md | Improves template spacing and adds mention of bun run pr. |
Suppressed comments (1)
.github/pull_request_template.md:25
- The reviewer checklist item is blank, so it’s easy to miss what reviewers are expected to verify. A short placeholder makes the template clearer and more actionable.
## Reviewer manual testing checklist
- [ ]
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| "format": "prettier --write .", | ||
| "test:unit": "vitest", | ||
| "test": "npm run test:unit -- --run" | ||
| "test": "npm run test:unit -- --run", |
There was a problem hiding this comment.
sveltekit scaffolding uses npm by default for tests even though project was created using bunx sv satcode. leaving it as is because of this and not using bun for that.
| - name: Set up Bun | ||
| uses: oven-sh/setup-bun@v2 | ||
| with: | ||
| bun-version: latest |
There was a problem hiding this comment.
fair but i think ill rather leave this like this and let ci break and tell us automatically if a new semver change breaks our pr ci
| ## Changes made | ||
| - | ||
|
|
||
| - |
There was a problem hiding this comment.
again with this nonsense you said this the last time already on another pr. bad copilot!
| Or run them all in one go: | ||
|
|
||
| ```sh | ||
| bun run pr | ||
| ``` | ||
|
|
||
| CI runs the same checks (all except `format`) on every pull request. |
There was a problem hiding this comment.
nope its stated in this pr that this is just a follow up fixup pr for the commit that got into the main accidentally
Summary
formatting for commit 4cd5 that was accidentally pushed to main directly without pr. changed ci, docs and scripts. ci are unlimited mins per month for public repos. 2000 minutes per month for private repos.
Related issue
Closes #12
Changes made
Author checklist
bun run format- code has been formattedbun run lint- no prettier/eslint errorsbun run check- no svelte-check/TypeScript errorsbun run test- all tests passbun run build- build succeedsReviewer manual testing checklist