Skip to content

ci: move from aqora-cli to aqora for pypi#188

Merged
jpopesculian merged 1 commit into
mainfrom
jpop/pypi-aqora
May 29, 2026
Merged

ci: move from aqora-cli to aqora for pypi#188
jpopesculian merged 1 commit into
mainfrom
jpop/pypi-aqora

Conversation

@jpopesculian
Copy link
Copy Markdown
Contributor

@jpopesculian jpopesculian commented Apr 24, 2026

Summary by CodeRabbit

  • Refactor
    • Renamed package distribution from aqora-cli to aqora. Updated module imports and namespace references throughout the codebase, including Python packages, Rust bindings, entry points, and templates.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 24, 2026

Warning

Review limit reached

@jpopesculian, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 31 minutes and 16 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 49a552bc-d509-42f9-93fd-46ccbf5a7043

📥 Commits

Reviewing files that changed from the base of the PR and between 670b05f and d8532cd.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (20)
  • .github/workflows/cd.yaml
  • .gitignore
  • README.md
  • docker/Dockerfile.kubimo
  • pyproject.toml
  • python/aqora/__init__.py
  • python/aqora/_aqora.pyi
  • python/aqora/fsspec.py
  • python/aqora/notebook.py
  • python/aqora/py.typed
  • python/aqora/pyarrow.py
  • runner/src/python.rs
  • src/commands/install.rs
  • src/python_module.rs
  • template/assets/dataset_marimo/pyproject.toml.hbs
  • template/assets/dataset_marimo/readme.py.hbs
  • template/assets/use_case/pyproject.toml.hbs
  • template/assets/use_case/use_case/use_case.py
  • test/test_aqora_client.py
  • test/test_aqorafs.py
📝 Walkthrough

Walkthrough

This PR systematically renames the package distribution from aqora-cli to aqora and updates the Rust extension module from _aqora_cli to _aqora. All imports, build metadata, installation references, generated templates, and tests are updated to use the new naming scheme.

Changes

Package rename: aqora-cli → aqora

Layer / File(s) Summary
Build metadata and entry points
pyproject.toml
Project distribution name, fsspec entry point, console script target, and Rust extension module name are all updated to use aqora instead of aqora-cli and _aqora instead of _aqora_cli.
Rust/PyO3 module export
src/python_module.rs
PyO3 module entrypoint is renamed from _aqora_cli to _aqora, the Client class is registered under module aqora, and exception wiring targets the aqora package.
Python package initialization and re-export
python/aqora/__init__.py
Package imports and re-exports symbols from ._aqora instead of _aqora_cli, and module docstring is sourced from the updated binding.
Python submodule imports and client configuration
python/aqora/fsspec.py, python/aqora/notebook.py, python/aqora/pyarrow.py
All submodules import Client from ._aqora, notebook cache directories and module name prefixes are updated to use aqora instead of aqora_cli, and AqoraFileSystem is imported from the new fsspec module path.
Installation and virtual environment setup
runner/src/python.rs, src/commands/install.rs
Virtual environment bootstrap and installation scripts request the aqora[venv] package extra instead of aqora-cli[venv].
Generated project templates
template/assets/dataset_marimo/pyproject.toml.hbs, template/assets/dataset_marimo/readme.py.hbs, template/assets/use_case/pyproject.toml.hbs, template/assets/use_case/use_case/use_case.py
Project templates declare dependencies on aqora instead of aqora-cli and import from aqora modules instead of aqora_cli modules.
Test imports
test/test_aqora_client.py, test/test_aqorafs.py
Tests import Client, ClientError, and dataset from aqora and aqora.pyarrow instead of aqora_cli.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 From aqora-cli to aqora we spring,
With _aqora bindings that sweetly ring,
Through templates and tests, a naming refine,
Each import renamed in perfect design,
The package hops forward, more graceful and lean! 🌟

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 27.27% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: renaming the PyPI package distribution from 'aqora-cli' to 'aqora' across all related modules, imports, and configurations.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jpop/pypi-aqora

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 and usage tips.

@jpopesculian jpopesculian force-pushed the jpop/pypi-aqora branch 3 times, most recently from 82c4b9e to 5739118 Compare May 29, 2026 11:17
@jpopesculian jpopesculian merged commit 2fcec11 into main May 29, 2026
15 of 17 checks passed
@jpopesculian jpopesculian deleted the jpop/pypi-aqora branch May 29, 2026 11:48
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