Skip to content

tests/release: replace submodules with version-pinned conformance fetch; adopt v0.0.1#24

Merged
fruwe merged 1 commit into
mainfrom
de-submodule
Jun 30, 2026
Merged

tests/release: replace submodules with version-pinned conformance fetch; adopt v0.0.1#24
fruwe merged 1 commit into
mainfrom
de-submodule

Conversation

@fruwe

@fruwe fruwe commented Jun 30, 2026

Copy link
Copy Markdown
Member

Closes #20.

Removes the git submodules (which have been painful) in favor of fetching the public conformance repo at the matching release tag.

Changes

  • Drop both submodules (vendor/harel, vendor/harel-conformance, .gitmodules).
  • A pytest fixture (tests/conftest.py) clones fruwehq/harel-conformance at v<version> (falls back to main until the tag exists) into a gitignored .cache/, reused across runs. HAREL_CONFORMANCE_DIR overrides with a local checkout for offline work; if the suite can't be obtained, conformance tests skip instead of erroring.
  • Schema-drift test fetches harel's schema at the matching tag by raw URL (HAREL_SPEC_DIR override), skipping when offline.
  • Version → 0.0.1 (the synchronized version across all fruwehq harel repos); README declares "implements the harel spec v0.0.1."
  • CI no longer checks out submodules.

Verified

Clean .cache removed, then pytest re-cloned the suite and ran 137 tests green (incl. the URL-fetched schema-drift check); ruff + mypy clean.

The v0.0.1 tags are cut in lockstep across the three repos once the 0.0.1 content is merged everywhere (tracked in #23).

…ch; adopt v0.0.1

Drop both git submodules. The conformance suite is fetched from
fruwehq/harel-conformance at the release tag matching this package's version
(falling back to main until the tag exists) into a gitignored .cache/ by a pytest
fixture; HAREL_CONFORMANCE_DIR overrides with a local checkout for offline work.
The schema-drift test fetches the spec schema by raw URL (HAREL_SPEC_DIR override),
skipping when offline. Set the synchronized version to 0.0.1 and declare the
implemented spec version in the README. CI no longer checks out submodules.

Closes #20.
@fruwe fruwe merged commit c55d831 into main Jun 30, 2026
1 check passed
@fruwe fruwe deleted the de-submodule branch June 30, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests/release: replace submodules with version-pinned conformance fetch; adopt v0.0.1

1 participant