chore: sync workflow templates - #6007
Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🚫 Review skipped — only excluded labels are configured. (7)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
Closing this generated delivery as no longer current. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3c8bdcabf0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| low_coverage = [f for f in all_files if f["coverage"] < low_threshold][:limit] | ||
|
|
||
| return hotspots, low_coverage | ||
| return hotspots, low_coverage, foreign |
There was a problem hiding this comment.
Preserve the existing two-value hotspot API
When the coverage-trend tests run, the existing callers in tests/test_coverage_trend.py still unpack _get_hotspots() into two variables, so returning a third value raises ValueError: too many values to unpack and blocks the test suite before its assertions execute. Update every caller and test to accept the foreign-file result, or preserve the previous return shape and expose that result separately.
Useful? React with 👍 / 👎.
|
Closing this generated delivery as no longer current. |
5 similar comments
|
Closing this generated delivery as no longer current. |
|
Closing this generated delivery as no longer current. |
|
Closing this generated delivery as no longer current. |
|
Closing this generated delivery as no longer current. |
|
Closing this generated delivery as no longer current. |
Automated sync from stranske/Workflows Template hash: 25782af9864e Changes synced from sync-manifest.yml Consumer-sync plan ID: sha256:25782af9864ed8fea0dbe923cf303e0cc14e72268afa8e83a47f7ac5c85954e1 Plan scope: source-delta Scope base SHA: 1f060bc6bd7aea61dd974e261506e529ff0f7aa3 Source commit: 2ca9251a5e17a7e69e5f41e2d439aa8b189d8927 Canary evidence JSON (base64): eyJzY2hlbWEiOiJ3b3JrZmxvd3MuY29uc3VtZXItc3luYy1jYW5hcnktZXZpZGVuY2UvdjEiLCJ2ZXJzaW9uIjoxLCJyZXN1bHRzIjpbeyJyZXBvIjoic3RyYW5za2UvVHJhdmVsLVBsYW4tUGVybWlzc2lvbiIsInBsYW5faWQiOiJzaGEyNTY6MjU3ODJhZjk4NjRlZDhmZWEwZGJlOTIzY2YzMDNlMGNjMTRlNzIyNjhhZmE4ZTgzYTQ3ZjdhYzVjODU5NTRlMSIsInBsYW5fc2NvcGUiOiJzb3VyY2UtZGVsdGEiLCJzY29wZV9iYXNlX3NoYSI6IjFmMDYwYmM2YmQ3YWVhNjFkZDk3NGUyNjE1MDZlNTI5ZmYwZjdhYTMiLCJzb3VyY2VfY29tbWl0IjoiMmNhOTI1MWE1ZTE3YTdlNjllNWY0MWUyZDQzOWFhOGIxODlkODkyNyIsInByIjoxNDc4LCJoZWFkX3NoYSI6IjJhODY4OWFkOTBhMmNjYzQ5OWIwNDdjYjYxOTA4OGQxMzYzNDg4NDAiLCJldmlkZW5jZV9zb3VyY2UiOiJvcGVuLWNhbmRpZGF0ZSIsInJlcXVpcmVkX2NoZWNrX3N0YXRlIjoic3VjY2VzcyIsImFjdGl2ZV9yZXZpZXdfdGhyZWFkX2NvdW50IjowfSx7InJlcG8iOiJzdHJhbnNrZS90cmlwLXBsYW5uZXIiLCJwbGFuX2lkIjoic2hhMjU2OjI1NzgyYWY5ODY0ZWQ4ZmVhMGRiZTkyM2NmMzAzZTBjYzE0ZTcyMjY4YWZhOGU4M2E0N2Y3YWM1Yzg1OTU0ZTEiLCJwbGFuX3Njb3BlIjoic291cmNlLWRlbHRhIiwic2NvcGVfYmFzZV9zaGEiOiIxZjA2MGJjNmJkN2FlYTYxZGQ5NzRlMjYxNTA2ZTUyOWZmMGY3YWEzIiwic291cmNlX2NvbW1pdCI6IjJjYTkyNTFhNWUxN2E3ZTY5ZTVmNDFlMmQ0MzlhYThiMTg5ZDg5MjciLCJwciI6MTc2NiwiaGVhZF9zaGEiOiI0MDZlMGFjNzQ5MTIyMmEzZGVhZmE5YjBkYjNmYmMwNzM4NTU3ZWJlIiwiZXZpZGVuY2Vfc291cmNlIjoib3Blbi1jYW5kaWRhdGUiLCJyZXF1aXJlZF9jaGVja19zdGF0ZSI6InN1Y2Nlc3MiLCJhY3RpdmVfcmV2aWV3X3RocmVhZF9jb3VudCI6MH0seyJyZXBvIjoic3RyYW5za2UvUG9ydGFibGUtQWxwaGEtRXh0ZW5zaW9uLU1vZGVsIiwicGxhbl9pZCI6InNoYTI1NjoyNTc4MmFmOTg2NGVkOGZlYTBkYmU5MjNjZjMwM2UwY2MxNGU3MjI2OGFmYThlODNhNDdmN2FjNWM4NTk1NGUxIiwicGxhbl9zY29wZSI6InNvdXJjZS1kZWx0YSIsInNjb3BlX2Jhc2Vfc2hhIjoiMWYwNjBiYzZiZDdhZWE2MWRkOTc0ZTI2MTUwNmU1MjlmZjBmN2FhMyIsInNvdXJjZV9jb21taXQiOiIyY2E5MjUxYTVlMTdhN2U2OWU1ZjQxZTJkNDM5YWE4YjE4OWQ4OTI3IiwicHIiOjIyNDQsImhlYWRfc2hhIjoiYmE0ZDk5NDM3NTk1YTc4MjAwYjNlNGRhYmRiMzlmMWVjMDg4MzI0MCIsImV2aWRlbmNlX3NvdXJjZSI6Im9wZW4tY2FuZGlkYXRlIiwicmVxdWlyZWRfY2hlY2tfc3RhdGUiOiJzdWNjZXNzIiwiYWN0aXZlX3Jldmlld190aHJlYWRfY291bnQiOjB9XX0=
3c8bdca to
9b78e52
Compare
|
Closing this generated delivery as no longer current. |
|
Closing this generated delivery as no longer current. |
Sync Summary
Files Updated
Review Checklist
Source: stranske/Workflows
Source SHA:
2ca9251a5e17a7e69e5f41e2d439aa8b189d8927Template hash:
25782af9864eConsumer-sync plan ID:
sha256:25782af9864ed8fea0dbe923cf303e0cc14e72268afa8e83a47f7ac5c85954e1Plan scope:
source-deltaScope base SHA:
1f060bc6bd7aea61dd974e261506e529ff0f7aa3Sync phase:
promoteSync branch:
sync/workflows-deliveryConsumer repo:
stranske/Trend_Model_ProjectManifest:
.github/sync-manifest.ymlautofix: false