Skip to content

Rename PyPI package to cpex-rate-limiter and update CLAUDE.md#4

Merged
jonpspri merged 1 commit into
mainfrom
rename-pypi-package
Apr 1, 2026
Merged

Rename PyPI package to cpex-rate-limiter and update CLAUDE.md#4
jonpspri merged 1 commit into
mainfrom
rename-pypi-package

Conversation

@jonpspri
Copy link
Copy Markdown
Collaborator

@jonpspri jonpspri commented Apr 1, 2026

Summary

  • Rename PyPI package from cpex-rate-limiter-plugin to cpex-rate-limiter
  • Expand CLAUDE.md with repository structure, build/test commands, DCO sign-off requirement, and coding conventions

Test plan

  • pip install cpex-rate-limiter works after next publish
  • CI passes on all platforms

- Rename PyPI package from cpex-rate-limiter-plugin to cpex-rate-limiter
- Expand CLAUDE.md with repository structure, build/test commands,
  and coding conventions

Signed-off-by: Jonathan Springer <jps@s390x.com>
Copilot AI review requested due to automatic review settings April 1, 2026 05:19
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates packaging metadata and contributor/agent documentation for the repository’s plugin workflow, primarily to reflect a new PyPI distribution name for the rate limiter plugin.

Changes:

  • Renames the rate limiter plugin’s PyPI package from cpex-rate-limiter-plugin to cpex-rate-limiter in pyproject.toml.
  • Expands repository guidance (structure, build/test commands, conventions, DCO sign-off) in documentation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
rate_limiter/pyproject.toml Updates the Python distribution name to the new cpex-rate-limiter package name.
AGENTS.md Adds expanded repo structure and build/test/conventions guidance (but currently appears to duplicate/overlap with CLAUDE.md).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AGENTS.md
Comment on lines 2 to +5

## Git

- All commits must be signed.
- All commits must include a DCO sign-off line. Always use `git commit -s` (or pass `-s` when committing).
Copy link

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is named AGENTS.md but its top-level heading says # CLAUDE.md, and the PR description says it updates CLAUDE.md (the repo also has a separate CLAUDE.md). Please align the filename/heading and PR description—either move these instructions into CLAUDE.md (and keep AGENTS.md agent-specific), or update the heading to # AGENTS.md and adjust the PR metadata accordingly to avoid confusion/duplication.

Copilot uses AI. Check for mistakes.
Comment thread AGENTS.md
## Git

- All commits must be signed.
- All commits must include a DCO sign-off line. Always use `git commit -s` (or pass `-s` when committing).
Copy link

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The wording here could be read as only requiring DCO sign-off, but other repo docs mention “Git with commit signing configured” (cryptographic commit signing). Please clarify whether the project requires (a) DCO sign-off only, (b) GPG/Sigstore commit signatures as well, or (c) both, and make the terminology consistent across docs.

Suggested change
- All commits must include a DCO sign-off line. Always use `git commit -s` (or pass `-s` when committing).
- All commits must be cryptographically signed (for example, using GPG or Sigstore with Git commit signing configured) **and** must include a DCO sign-off line. Always use `git commit -S -s` (or equivalent flags when committing).

Copilot uses AI. Check for mistakes.
@jonpspri jonpspri merged commit f7e0641 into main Apr 1, 2026
20 checks passed
@jonpspri jonpspri deleted the rename-pypi-package branch April 1, 2026 05:24
lucarlig pushed a commit that referenced this pull request Apr 1, 2026
- Rename PyPI package from cpex-rate-limiter-plugin to cpex-rate-limiter
- Expand CLAUDE.md with repository structure, build/test commands,
  and coding conventions

Signed-off-by: Jonathan Springer <jps@s390x.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
lucarlig pushed a commit that referenced this pull request Apr 1, 2026
- Rename PyPI package from cpex-rate-limiter-plugin to cpex-rate-limiter
- Expand CLAUDE.md with repository structure, build/test commands,
  and coding conventions

Signed-off-by: Jonathan Springer <jps@s390x.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
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.

2 participants