Skip to content

Improve README and CONTRIBUTING documentation#25

Merged
merlimat merged 1 commit intooxia-db:mainfrom
merlimat:fix/docs-readme-contributing
Apr 16, 2026
Merged

Improve README and CONTRIBUTING documentation#25
merlimat merged 1 commit intooxia-db:mainfrom
merlimat:fix/docs-readme-contributing

Conversation

@merlimat
Copy link
Copy Markdown
Contributor

Summary

README.md — rewritten:

  • Fixed h2/h1 tag mismatch
  • Added install instructions (pip install oxia)
  • Added quick-start code snippet showing put/get/list/delete with context manager
  • Added feature list covering all supported operations
  • Fixed CI badge to point to the Python client repo (was pointing to the server)
  • Added Python 3.10+ requirement
  • Removed broken empty [guide]() link
  • Rewrote subtitle to describe the client, not Oxia itself

CONTRIBUTING.md — rewritten for Python:

  • Replaced "golang project" / make test / make lint with uv run pytest
  • Fixed clone URL from oxia to oxia-client-python
  • Added development setup section (uv, Docker for testcontainers, proto regen)
  • Documented DCO sign-off requirement

__init__.py — fixed usage example:

  • Removed oxia:// scheme prefix (not used by the client)
  • Added context manager pattern
  • Changed value to bytes literal

README.md:
- Fix h2/h1 tag mismatch
- Add install instructions, quick-start code snippet, feature list
- Fix badge to point to the Python client repo
- Add Python version requirement
- Remove broken empty [guide]() link
- Rewrite subtitle to describe the client, not Oxia itself

CONTRIBUTING.md:
- Replace "golang project" with Python-specific instructions
- Replace make test/lint with uv run pytest
- Fix clone URL from oxia to oxia-client-python
- Add development setup section (uv, Docker, proto regen)
- Document DCO sign-off requirement

__init__.py:
- Fix usage example: remove oxia:// scheme prefix, use context
  manager, use bytes value

Signed-off-by: Matteo Merli <mmerli@apache.org>
@merlimat merlimat merged commit 9b8f4bb into oxia-db:main Apr 16, 2026
6 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