Skip to content

fix: use absolute README links for PyPI - #23

Merged
UlloaSP merged 3 commits into
mainfrom
develop
Oct 19, 2026
Merged

UlloaSP merged 3 commits into
mainfrom
develop

Conversation

@UlloaSP

@UlloaSP UlloaSP commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • README links to docs, SECURITY, LICENSE, AGENTS and publishing were relative, so they broke on PyPI. Site badges also pointed at source folders instead of the deployed sites.
  • Links now point at the deployed landing/docs/playground and GitHub blobs; a new test rejects any non-https:// README link.

Verification

  • New test fails on the old README and passes with the fix
  • uv run pytest: 1768 passed, 91.96% coverage; ruff, pyright clean; twine check --strict passed
  • Crawled the live sites (30 pages, 278 links) and checked 53 hardcoded source/README URLs: 0 broken

Add the landing, documentation and playground sites with shared eshkra
branding, tested builds and Cloudflare deployment from main. Publish to
PyPI automatically when a push to main changes project.version, using the
CI-built distributions and the pypi_env token. Move schema inference to
mlschema 0.3 and prepare version 0.2.0.
PyPI renders the README without the repository, so relative links to docs,
LICENSE and site folders broke. Point them at the deployed sites and GitHub,
and test that every README link is absolute.
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b195ed59-3d41-4426-ba6d-ba9db3fc8d11
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@UlloaSP
UlloaSP merged commit 676f46e into main Oct 19, 2026
21 checks passed
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