Skip to content

chore: enhance CI with linting and whitespace checks in GitHub Actions#579

Merged
akrem-chabchoub merged 2 commits intomasterfrom
fix/ci
Apr 14, 2026
Merged

chore: enhance CI with linting and whitespace checks in GitHub Actions#579
akrem-chabchoub merged 2 commits intomasterfrom
fix/ci

Conversation

@akrem-chabchoub
Copy link
Copy Markdown
Contributor

@akrem-chabchoub akrem-chabchoub commented Apr 10, 2026

summary

Align beekeeper CI with ethersphere/bee: conventional commit lint on PRs, PR title lint, and extra quality checks (whitespace + go mod tidy). Splits lint onto Ubuntu only and keeps build/test on the OS matrix.

changes

  • add commitlint.config.js at repo root (@commitlint/config-conventional rules, header max length 98)
  • add .github/workflows/pr-title.yml — lint PR title on opened, synchronize, reopened, edited
  • update .github/workflows/go.yml:
    • pull_request with paths-ignore: '**/*.md' for the Go workflow
    • new lint job: full history checkout, wagoid/commitlint-github-action@v5, golangci-lint, make vet, make check-whitespace, katexochen/go-tidy-check@v2
    • build job: matrix build + test only (shallow checkout)
  • add check-whitespace target to Makefile (same idea as bee)
  • fix existing trailing whitespace / EOF whitespace issues so the new check passes

why

Keeps commit and merge messages consistent across swarm repos and catches bad titles/commits before merge.

notes

  • Markdown-only PRs skip the Go workflow but still get PR title lint.
  • Update branch protection if you want Lint and Lint PR Title required.

Refs: ethersphere/bee go.yml, pr-title.yml, commitlint.config.js.

@akrem-chabchoub akrem-chabchoub self-assigned this Apr 10, 2026
@akrem-chabchoub akrem-chabchoub marked this pull request as ready for review April 10, 2026 13:27
@akrem-chabchoub akrem-chabchoub changed the title chore: enhance CI with linting and whitespace checks in GitHub Actions Enhance CI with linting and whitespace checks in GitHub Actions Apr 10, 2026
@akrem-chabchoub akrem-chabchoub changed the title Enhance CI with linting and whitespace checks in GitHub Actions chore: enhance CI with linting and whitespace checks in GitHub Actions Apr 10, 2026
@akrem-chabchoub akrem-chabchoub added this to the 2026 milestone Apr 14, 2026
@akrem-chabchoub akrem-chabchoub merged commit 322d888 into master Apr 14, 2026
6 of 7 checks passed
@akrem-chabchoub akrem-chabchoub deleted the fix/ci branch April 14, 2026 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhance workflows: commit lint, PR title lint, and optional checks

4 participants