Skip to content

Clarify Coveralls CI behavior#34

Closed
devnexus-automation[bot] wants to merge 1 commit into
mainfrom
codex/coveralls-pr-semantics
Closed

Clarify Coveralls CI behavior#34
devnexus-automation[bot] wants to merge 1 commit into
mainfrom
codex/coveralls-pr-semantics

Conversation

@devnexus-automation
Copy link
Copy Markdown
Contributor

Summary

  • keep Coveralls reporting on pull requests so coverage deltas remain visible
  • make the upload step explicitly depend on successful tests
  • keep the parallel finalizer running after the matrix, and name it as a finalizer
  • switch from deprecated path-to-lcov to current file/format inputs

Verification

  • git diff --check
  • ruby -e 'require "yaml"; doc = YAML.load_file(".github/workflows/ci.yml"); raise "missing jobs" unless doc["jobs"]; puts "yaml ok"'\n- workflow structure check for Coveralls upload/finalizer keys\n\nSmalltalkCI was not run locally; this is a GitHub Actions YAML-only change.

Co-authored-by: Codex <codex@openai.com>
@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/coveralls-pr-semantics branch from ab1cbc2 to 759d96c Compare May 30, 2026 20:52
@Gabriel-Darbord Gabriel-Darbord deleted the codex/coveralls-pr-semantics branch May 30, 2026 20:58
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.

1 participant