Repository navigation
chore: remove the Python bindings, moved to ArcadeData/arcadedb-python-bindings - #10615
Conversation
|
Important Review skippedToo many files! This PR contains 156 files, which is 6 over the limit of 150. To get a review, reduce the PR to 150 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configuration
📒 Files selected for processing: 156
You can disable this status message by setting the
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. Comment |
|
This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again. |
Up to standards ✅🟢 Issues
|
Review: removal of the Python bindingsThis is a pure removal and I found nothing blocking. Checked
Minor, optional
LGTM, assuming CI is green. I did not build or run tests locally. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…n-bindings bindings/python moved, with its history, to https://github.com/ArcadeData/arcadedb-python-bindings (extracted at ee04bd2; nothing touched it here since). That repository now builds against arcadedata/arcadedb:latest and tests every new snapshot nightly. Removes the directory, its two workflows, and every reference to it: the paths-ignore filters in mvn-test and mvn-deploy, the Meterian exclude, the Dependabot uv entry, actionlint ignores, the Codacy test exclusion, the REUSE annotation, the four pre-commit hooks scoped to it, and the gitignore allowlist entries for its generated files. The README points at the new repo. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
19138db to
0c5de66
Compare
|
Rebased onto |
|
Review: this PR only removes the Python bindings and the CI and config references to them. It adds no code or dependencies, so I see no security concerns.
Looks good to merge once CI is green. |
Retire the upstream sync (upstream deleted bindings/python in ArcadeData#10615)
The Python bindings (
arcadedb-embedded) now live in their own repository, ArcadeData/arcadedb-python-bindings. This PR removes them from the monorepo.ee04bd202abce0183c6263a3c4d8a61a4d526960, andbindings/pythonhas not changed here since.arcadedata/arcadedb:latest, with the package version read from the engine jar inside the image. They follow each new snapshot automatically and are tested nightly on Linux, macOS and Windows.main.Removed here
bindings/python/.github/workflows/test-python-bindings.ymlandtest-python-examples.ymlpaths-ignoreentries inmvn-test.ymlandmvn-deploy.yml--exclude-foldersuventry and the actionlint ignorescheck-gitignore-src-paths.shNot touched:
e2e-python/(server wire tests, unrelated to the bindings) and historical documents underdocs/.Checked locally
actionlint.github/scripts/check-gitignore-src-paths.sh.github/scripts/tests/test-ci-scripts.sh(174/174)🤖 Generated with Claude Code