Skip to content

ci: add kit contract tests action - #5190

Merged
josephjclark merged 4 commits into
mainfrom
kit-contract-tests
Sep 17, 2026
Merged

josephjclark merged 4 commits into
mainfrom
kit-contract-tests

Conversation

@doc-han

@doc-han doc-han commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Description

Adds a CI action for checking contract tests between lightning(current branch) and kit.

Validation steps

  1. (How can a reviewer validate your work?)

Additional notes for the reviewer

  1. (Is there anything else the reviewer should know or look out for?)

AI Usage

Please disclose whether you've used AI anywhere in this PR (it's cool, we just
want to know!):

  • I have used Claude Code
  • I have used another model
  • I have not used AI

You can read more details in our
Responsible AI Policy

Pre-submission checklist

  • I have performed an AI review of my code (we recommend using /review
    with Claude Code)
  • I have implemented and tested all related authorization policies.
    (e.g., :owner, :admin, :editor, :viewer)
  • I have updated the changelog.
  • I have ticked a box in "AI usage" in this PR

@github-project-automation github-project-automation Bot moved this to New Issues in Core Sep 16, 2026
@github-actions

Copy link
Copy Markdown

Security Review ✅

  • S0 (project scoping): N/A, PR only adds a GitHub Actions workflow file (.github/workflows/kit-contract.yml:1) with no application data access.
  • S1 (authorization): N/A, no web-layer actions, controllers, LiveView events, or policy modules are added or modified.
  • S2 (audit trail): N/A, no writes to project or instance configuration resources are introduced.

@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.9%. Comparing base (ffa1d38) to head (c8beeb9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #5190     +/-   ##
=======================================
- Coverage   90.9%   90.9%   -0.1%     
=======================================
  Files        425     425             
  Lines      21156   21156             
=======================================
- Hits       19241   19230     -11     
- Misses      1915    1926     +11     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@doc-han doc-han changed the title ci: kit contract tests ci: add kit contract tests action Sep 16, 2026
@josephjclark
josephjclark merged commit 98665f1 into main Sep 17, 2026
8 checks passed
@josephjclark
josephjclark deleted the kit-contract-tests branch September 17, 2026 09:48
@github-project-automation github-project-automation Bot moved this from New Issues to Done in Core Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants