Chore/update to timely beliefs v3.5.5 - #2148
Conversation
Signed-off-by: F.N. Claessen <claessen@seita.nl>
…ors.InvalidSchemaName) schema "np" does not exist` Signed-off-by: F.N. Claessen <claessen@seita.nl>
Signed-off-by: F.N. Claessen <claessen@seita.nl>
…n upgrades Signed-off-by: F.N. Claessen <claessen@seita.nl>
Signed-off-by: F.N. Claessen <claessen@seita.nl>
Signed-off-by: F.N. Claessen <claessen@seita.nl>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Signed-off-by: F.N. Claessen <claessen@seita.nl>
…an diff Signed-off-by: F.N. Claessen <claessen@seita.nl>
Signed-off-by: F.N. Claessen <claessen@seita.nl>
Context: - PR #2148 failed with uv sync --locked after uv lock --upgrade - Fork markers from impossible platform combos caused coverage check to fail Change: - Added pattern for diagnosing and fixing uv fork marker errors Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
…d flag The uv.lock fork markers included impossible platform combinations (e.g. os_name == 'nt' and sys_platform == 'darwin') that uv could not simplify when checking coverage against the full supported environment space. Adding [tool.uv] environments restricts resolution to the three real platforms (linux, darwin, win32), ensuring the fork markers perfectly cover the declared environments and uv sync --locked succeeds. Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Context: - File had grown to ~31KB with duplicated content - File size limit is 30KB; file was at 30,997 bytes Change: - Removed 'Integration with Lead' block (duplicates Lead's session-close checklist) - Removed 'Performance Impact' of db fixtures (covered by the decision tree) - Removed '404 vs 422' code examples (covered by surrounding prose) - Removed 'Test Execution Workflow (CRITICAL)' section (commands exist in other sections) Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Context: - PR #2148 failed with uv sync --locked after uv lock --upgrade - Fork markers from impossible platform combos caused coverage check to fail Change: - Added pattern for diagnosing and fixing uv fork marker errors Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Code review suggested documenting why the platform restrictions are necessary to help future maintainers understand the fix. Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
|
Comment I requested from copilot, but it had trouble posting on this PR (retrieved from session logs): Fix for failing CI workflowsRoot cause: The warning message was: And the sync failure was: This affected both the Fix: Added [tool.uv]
environments = [
"sys_platform == 'linux'",
"sys_platform == 'darwin'",
"sys_platform == 'win32'",
]The fix is available in commits:
These commits are on the copilot branch git fetch origin copilot/choreupdate-to-timely-beliefs-v3-5-5
git cherry-pick f7dd9228
git pushNote: The |
Signed-off-by: F.N. Claessen <claessen@seita.nl>
Signed-off-by: F.N. Claessen <claessen@seita.nl>
Description
documentation/changelog.rstLook & Feel
How to test
...
Further Improvements
...
Related Items
Closes #2078.