Skip to content

feat(ci): add Lua coverage reporting and badge pipeline#26

Merged
levleontiev merged 4 commits intomainfrom
feat/issue-24-lua-coverage
Mar 14, 2026
Merged

feat(ci): add Lua coverage reporting and badge pipeline#26
levleontiev merged 4 commits intomainfrom
feat/issue-24-lua-coverage

Conversation

@levleontiev
Copy link
Contributor

Summary

  • replace misleading Python E2E coverage path with Lua coverage via luacov
  • generate coverage summary and badge JSON payloads from CI
  • publish badge payload through GitHub Pages and wire README to the Pages-backed shield
  • add a non-generated Lua coverage regression gate so autogenerated files do not hide coverage drops

Verification

  • docker run --rm --user "$(id -u):$(id -g)" -v "$PWD:/work" -w /work fairvisor-test-ci bin/ci/run_lua_coverage.sh
  • bin/ci/build_lua_coverage_badge.sh
  • bash -n bin/ci/run_lua_coverage.sh bin/ci/build_lua_coverage_badge.sh
  • LUA_COVERAGE_MIN_NON_GENERATED=82.29 bin/ci/build_lua_coverage_badge.sh fails as expected

Notes

  • Current local metrics: 98.47% total, 82.28% non-generated
  • Badge/gate use the non-generated metric because src/fairvisor/generated/bot_automaton.lua would otherwise dominate the denominator
  • Draft PR for now; not calling review until the full required test/CI path is confirmed

Refs #24

@levleontiev levleontiev marked this pull request as ready for review March 14, 2026 14:38
@levleontiev levleontiev merged commit 0cd16c0 into main Mar 14, 2026
8 checks passed
@levleontiev levleontiev deleted the feat/issue-24-lua-coverage branch March 14, 2026 14:40
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.

2 participants