Repository navigation
chore: [DO NOT MERGE] CI trial of #1591 merged into #1589 - #1592
Closed
gabrielseco wants to merge 4 commits into
Closed
gabrielseco wants to merge 4 commits into
gabrielseco wants to merge 4 commits into
Conversation
Pins Portugal to its latest contract details version in the example app, documents v4 to v9 in the country changelog and updates the PRT e2e for the new required fields. v7 and later are jsf v1 schemas, so the e2e now asserts the jsf v1 path. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… file changed-countries.ts fell back to "all" whenever a PR touched any file besides the pins file, so a pin bump that also updated docs or e2e (for example #1589) ran every country. Only the files the workflow already lists as inputs now widen the scope. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…non-inputs' into tmp/try-1591-into-1589
Contributor
|
Deploy preview for adp-cost-calculator ready!
Deployed with vercel-action |
Contributor
|
Deploy preview for remote-flows ready!
Deployed with vercel-action |
Contributor
📊 Coverage Report
|
| Metric | Current | Previous | Change | Status |
|---|---|---|---|---|
| Lines | 89.58% | 89.59% | -0.02% | 🔴 |
| Statements | 89.45% | 89.47% | -0.02% | 🔴 |
| Functions | 88.48% | 88.48% | 0% | ⚪ |
| Branches | 81.24% | 81.24% | 0% | ⚪ |
Detailed Breakdown
Lines Coverage
- Covered: 5396 / 6024
- Coverage: 89.58%
- Change: -0.02% (-1 lines)
Statements Coverage
- Covered: 5461 / 6105
- Coverage: 89.45%
- Change: -0.02% (-1 statements)
Functions Coverage
- Covered: 1428 / 1614
- Coverage: 88.48%
- Change: 0% (0 functions)
Branches Coverage
- Covered: 3351 / 4125
- Coverage: 81.24%
- Change: 0% (0 branches)
✅ Coverage check passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Throwaway draft that combines #1589 and #1591 so CI runs on their merge. Do not merge; close once checked.
Why
Check that the schema canary scoping from #1591 behaves correctly on #1589's PRT schema bump.
What changed
Toggle details
Branch of #1589 with the #1591 branch merged in. No other changes. The canary reports
allhere because the merge edits the canary scripts themselves.Screenshots
N/A
Related Resources
Testing
example/app in a browser🤖 Generated with Claude Code