-
Notifications
You must be signed in to change notification settings - Fork 0
feat: bounty creation ui formatting #32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,42 @@ | ||
| name: CI | ||
|
|
||
| on: | ||
| pull_request: | ||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
| group: ci-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: true | ||
|
|
||
| jobs: | ||
| checks: | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 15 | ||
| steps: | ||
| - name: Check out repository | ||
| uses: actions/checkout@v5 | ||
|
|
||
| - name: Set up Bun | ||
| uses: oven-sh/setup-bun@v2 | ||
| with: | ||
| bun-version: latest | ||
|
Comment on lines
+19
to
+22
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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 |
||
|
|
||
| - name: Install dependencies | ||
| run: bun install --frozen-lockfile | ||
|
|
||
| # Needed by the vitest browser-mode project (chromium via playwright) | ||
| - name: Install Playwright browsers | ||
| run: bunx playwright install chromium --with-deps | ||
|
|
||
| # `bun run format` is local-only; CI verifies formatting via prettier --check in `lint` | ||
| - name: Lint (prettier + eslint) | ||
| run: bun run lint | ||
|
|
||
| - name: Check (svelte-check + TypeScript) | ||
| run: bun run check | ||
|
|
||
| - name: Test | ||
| run: bun run test | ||
|
|
||
| - name: Build | ||
| run: bun run build | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -37,6 +37,14 @@ bun run test # unit tests | |
| bun run build # production build | ||
| ``` | ||
|
|
||
| Or run them all in one go: | ||
|
|
||
| ```sh | ||
| bun run pr | ||
| ``` | ||
|
|
||
| CI runs the same checks (all except `format`) on every pull request. | ||
|
Comment on lines
+40
to
+46
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nope its stated in this pr that this is just a follow up fixup pr for the commit that got into the main accidentally |
||
|
|
||
| ## License | ||
|
|
||
| This program is free software: you can redistribute it and/or modify it under the terms of the GNU Affero General Public License as published by the Free Software Foundation, either version 3 of the License, or any later version. | ||
| This program is free software: you can redistribute it and/or modify it under the terms of the GNU Affero General Public License as published by the Free Software Foundation, either version 3 of the License, or any later version. | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,7 +13,8 @@ | |
| "lint": "prettier --check . && eslint .", | ||
| "format": "prettier --write .", | ||
| "test:unit": "vitest", | ||
| "test": "npm run test:unit -- --run" | ||
| "test": "npm run test:unit -- --run", | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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. |
||
| "pr": "bun run format && bun run lint && bun run check && bun run test && bun run build" | ||
| }, | ||
| "devDependencies": { | ||
| "@eslint/js": "^10.0.1", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
again with this nonsense you said this the last time already on another pr. bad copilot!