Repository navigation
CI: run native parity on changes to shared tests and source - #95
Merged
Merged
Conversation
test-native.yml ran only for native paths, so a change to a test or source file shared by both backends skipped the native parity suite. PR #88 changed tests/test_log_redirect.py and main's native parity broke unseen until #92. The pull_request and push paths (one YAML anchor) now cover src/arcadedb/** and tests/**, plus the native paths it already covered. Docs outside those paths still do not trigger it. No job changed. ci.adoc states which paths trigger native parity and why. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
…moke_wheel.py changes (the native job uses them) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
test-native.ymlruns the native parity suite only when a native path changes. A change to a test or source file that both backends share therefore skipped it: PR #88 changedtests/test_log_redirect.py, and main's native parity broke unseen until #92 fixed it. This is OPEN-ITEMS #61.The
pathsfilter is one YAML anchor shared bypull_requestandpush, so both events change together. It now listsnative/**,src/arcadedb/**,tests/**,scripts/select_native_matrix.pyand.github/workflows/test-native.yml. The old entries forsrc/arcadedb/_native/**,tests/test_native_*.py,tests/conftest.pyandtests/test_select_native_matrix.pyare covered by the two wider globs, so they are gone. No job changed.The filter is a positive list, so docs outside those paths still do not trigger the workflow (docs/**, .adoc, CLAUDE.md, LICENSE).
tests/README.mdis undertests/**and does trigger it:tests/test_no_monorepo_wording.pyreads it, so a change there can break the suite.This PR changes
test-native.ymlitself, so its own run is one native leg (linux-x86_64, Python 3.12), about 1 to 2.5 hours. The JVM workflow ignores this PR (it only touches the native workflow and docs).docs/modules/ROOT/pages/development/ci.adocstates the trigger paths and the reason in its pull request bullet under Native build.Checks run locally: the YAML parses with
yaml.safe_load(bothpull_request.pathsandpush.pathsresolve to the same list), andscripts/check_no_embedded_name.py --root .passes.🤖 Generated with Claude Code