Skip to content

annotatedyaml: Add version 1.0.2 - #2313

Merged
luhenry merged 3 commits into
mainfrom
annotatedyaml
Sep 26, 2026
Merged

luhenry merged 3 commits into
mainfrom
annotatedyaml

Conversation

@luhenry

@luhenry luhenry commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Compiles two Cython modules (constructors, reference) from the package's own Python sources. Upstream publishes no riscv64 wheel.

Mirrors the build_wheels and test jobs of upstream's ci.yml.

Differs from upstream

  • REQUIRE_CYTHON=1 is passed into the build container - upstream's job-level env never reaches it on Linux.

Matrix: cp313/cp314/cp314t, manylinux and musllinux - requires-python >=3.13, same set upstream builds.

Testing

  • same as upstream, plus an import check that both modules load as .so - build_ext.py silently falls back to pure Python if compilation fails.
  • no coverage - upstream's pytest-cov addopts are not staged.

License: OK

Built on cp313; 119 passed, 1 skipped.

luhenry added a commit that referenced this pull request Sep 25, 2026
@github-actions

github-actions Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-26 08:14 UTC

No musllinux riscv64 PyYAML wheel exists, so the test install builds
it from sdist; the musllinux image has no libyaml headers, so PyYAML
silently falls back to pure Python and the tests' try_both_loaders /
try_both_dumpers fixtures fail on the missing CSafeLoader/CSafeDumper.
Install yaml-dev before testing on musllinux and set
PYYAML_FORCE_LIBYAML=1 so a missing libyaml fails the build instead.
@luhenry
luhenry merged commit 222f5a3 into main Sep 26, 2026
14 checks passed
@luhenry
luhenry deleted the annotatedyaml branch September 26, 2026 07:52
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.

1 participant