Skip to content

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

Open
akrem-chabchoub wants to merge 2 commits intomasterfrom
fix/ci
Open

chore: enhance CI with linting and whitespace checks in GitHub Actions#579
akrem-chabchoub wants to merge 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 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
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

1 participant