From 0e4ebe27ddb4b294e1cac482732abc23d85256d2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D7=A0=CF=85=CE=B1=CE=B7=20=D7=A0=CF=85=CE=B1=CE=B7=D1=95?= =?UTF-8?q?=CF=83=CE=B7?= Date: Sun, 15 Mar 2026 15:43:33 -0700 Subject: [PATCH 1/3] chore: clean up CI workflow naming and add lint jobs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Rename workflow files for consistent hyphenated naming: - commitlint.yml → commit-lint.yml - depreview.yml → dep-review.yml - reportcard.yml → report-card.yml - docs.yml → docs-deploy.yml - docs-test.yml → docs-build.yml (lint moved to own file) Add new standalone lint workflows: - docs-lint.yml runs docs::fmt-check - just-lint.yml runs just::fmt-check Add just::fmt-check to the test recipe and a ready recipe for local pre-commit formatting, linting, and codegen. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- .../{commitlint.yml => commit-lint.yml} | 0 .../{depreview.yml => dep-review.yml} | 0 .../{docs-test.yml => docs-build.yml} | 8 +++---- .../workflows/{docs.yml => docs-deploy.yml} | 0 .github/workflows/docs-lint.yml | 21 +++++++++++++++++++ .github/workflows/just-lint.yml | 20 ++++++++++++++++++ .../{reportcard.yml => report-card.yml} | 0 justfile | 11 ++++++++++ 8 files changed, 55 insertions(+), 5 deletions(-) rename .github/workflows/{commitlint.yml => commit-lint.yml} (100%) rename .github/workflows/{depreview.yml => dep-review.yml} (100%) rename .github/workflows/{docs-test.yml => docs-build.yml} (71%) rename .github/workflows/{docs.yml => docs-deploy.yml} (100%) create mode 100644 .github/workflows/docs-lint.yml create mode 100644 .github/workflows/just-lint.yml rename .github/workflows/{reportcard.yml => report-card.yml} (100%) diff --git a/.github/workflows/commitlint.yml b/.github/workflows/commit-lint.yml similarity index 100% rename from .github/workflows/commitlint.yml rename to .github/workflows/commit-lint.yml diff --git a/.github/workflows/depreview.yml b/.github/workflows/dep-review.yml similarity index 100% rename from .github/workflows/depreview.yml rename to .github/workflows/dep-review.yml diff --git a/.github/workflows/docs-test.yml b/.github/workflows/docs-build.yml similarity index 71% rename from .github/workflows/docs-test.yml rename to .github/workflows/docs-build.yml index aad24c7b0..46f41dabf 100644 --- a/.github/workflows/docs-test.yml +++ b/.github/workflows/docs-build.yml @@ -1,5 +1,5 @@ --- -name: Test Deploy to GitHub Pages +name: Docs Build on: pull_request: @@ -7,8 +7,8 @@ on: - main jobs: - test-deploy: - name: Test deployment + build: + name: Test build website runs-on: ubuntu-latest steps: - uses: actions/checkout@v6 @@ -17,7 +17,5 @@ jobs: uses: extractions/setup-just@v3 - name: Fetch justfiles run: just fetch - - name: Lint the website - run: just docs::fmt-check - name: Test build website run: just docs::build diff --git a/.github/workflows/docs.yml b/.github/workflows/docs-deploy.yml similarity index 100% rename from .github/workflows/docs.yml rename to .github/workflows/docs-deploy.yml diff --git a/.github/workflows/docs-lint.yml b/.github/workflows/docs-lint.yml new file mode 100644 index 000000000..7db4d47c1 --- /dev/null +++ b/.github/workflows/docs-lint.yml @@ -0,0 +1,21 @@ +--- +name: Docs Lint + +on: + push: + branches: ["main"] + pull_request: + branches: ["main"] + +jobs: + lint: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v6 + - uses: oven-sh/setup-bun@v2 + - name: Install just + uses: extractions/setup-just@v3 + - name: Fetch justfiles + run: just fetch + - name: Lint docs + run: just docs::fmt-check diff --git a/.github/workflows/just-lint.yml b/.github/workflows/just-lint.yml new file mode 100644 index 000000000..926ddeaed --- /dev/null +++ b/.github/workflows/just-lint.yml @@ -0,0 +1,20 @@ +--- +name: Just Lint + +on: + push: + branches: ["main"] + pull_request: + branches: ["main"] + +jobs: + lint: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v6 + - name: Install just + uses: extractions/setup-just@v3 + - name: Fetch justfiles + run: just fetch + - name: Lint justfiles + run: just just::fmt-check diff --git a/.github/workflows/reportcard.yml b/.github/workflows/report-card.yml similarity index 100% rename from .github/workflows/reportcard.yml rename to .github/workflows/report-card.yml diff --git a/justfile b/justfile index 9584690bd..ec25a72eb 100644 --- a/justfile +++ b/justfile @@ -3,6 +3,7 @@ # validates dependencies at parse time, which would fail when modules aren't loaded. mod? go '.just/remote/go.mod.just' mod? docs '.just/remote/docs.mod.just' +mod? just '.just/remote/just.mod.just' # --- Fetch --- @@ -13,6 +14,8 @@ fetch: curl -sSfL https://raw.githubusercontent.com/osapi-io/osapi-justfiles/refs/heads/main/go.just -o .just/remote/go.just curl -sSfL https://raw.githubusercontent.com/osapi-io/osapi-justfiles/refs/heads/main/docs.mod.just -o .just/remote/docs.mod.just curl -sSfL https://raw.githubusercontent.com/osapi-io/osapi-justfiles/refs/heads/main/docs.just -o .just/remote/docs.just + curl -sSfL https://raw.githubusercontent.com/osapi-io/osapi-justfiles/refs/heads/main/just.mod.just -o .just/remote/just.mod.just + curl -sSfL https://raw.githubusercontent.com/osapi-io/osapi-justfiles/refs/heads/main/just.just -o .just/remote/just.just # --- Top-level orchestration --- @@ -24,6 +27,7 @@ deps: # Run all tests test: linux-tune + just just::fmt-check just go::test # Generate code @@ -32,6 +36,13 @@ generate: just go::generate just docs::generate +# Format, lint, and generate before committing +ready: + just generate + just docs::fmt + just go::fmt + just go::vet + [linux] linux-tune: sudo sysctl -w net.ipv4.ping_group_range="0 2147483647" From 4b4e085af1bc438cc4d385f4ffae22d114c0b8e6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D7=A0=CF=85=CE=B1=CE=B7=20=D7=A0=CF=85=CE=B1=CE=B7=D1=95?= =?UTF-8?q?=CF=83=CE=B7?= Date: Sun, 15 Mar 2026 15:45:16 -0700 Subject: [PATCH 2/3] style: fix comment wrapping in justfile MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- justfile | 1 + 1 file changed, 1 insertion(+) diff --git a/justfile b/justfile index ec25a72eb..3d3c27e61 100644 --- a/justfile +++ b/justfile @@ -1,5 +1,6 @@ # Optional modules: mod? allows `just fetch` to work before .just/remote/ exists. # Recipes below use `just` subcommands instead of dependency syntax because just + # validates dependencies at parse time, which would fail when modules aren't loaded. mod? go '.just/remote/go.mod.just' mod? docs '.just/remote/docs.mod.just' From a46aa7c68a0ad79449b8dfca1545a8bfff700203 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D7=A0=CF=85=CE=B1=CE=B7=20=D7=A0=CF=85=CE=B1=CE=B7=D1=95?= =?UTF-8?q?=CF=83=CE=B7?= Date: Sun, 15 Mar 2026 15:53:53 -0700 Subject: [PATCH 3/3] fix: remove just::fmt-check from test recipe MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Justfile linting is handled by the standalone just-lint.yml workflow and should not block the Go test pipeline. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- justfile | 1 - 1 file changed, 1 deletion(-) diff --git a/justfile b/justfile index 3d3c27e61..53cd41d36 100644 --- a/justfile +++ b/justfile @@ -28,7 +28,6 @@ deps: # Run all tests test: linux-tune - just just::fmt-check just go::test # Generate code