Skip to content

Maintenance 2026-05#161

Merged
amotl merged 5 commits into
mainfrom
this-and-that-202605
May 16, 2026
Merged

Maintenance 2026-05#161
amotl merged 5 commits into
mainfrom
this-and-that-202605

Conversation

@amotl
Copy link
Copy Markdown
Member

@amotl amotl commented May 16, 2026

This and that.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 16, 2026

Review Change Stack

Warning

Rate limit exceeded

@amotl has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 56 minutes and 48 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, 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 have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9c234787-5978-4380-877d-905bb175a5c9

📥 Commits

Reviewing files that changed from the base of the PR and between 908dad0 and 073a771.

📒 Files selected for processing (5)
  • Makefile
  • epo_ops/models.py
  • pyproject.toml
  • setup.py
  • tests/helpers/api_helpers.py

Walkthrough

This PR updates project infrastructure and configurations: setup.py now uses automatic package discovery with find_packages, pyproject.toml reorganizes build-system, pytest, Ruff, and Versioningit sections, Makefile integrates pyproject-fmt formatting, and license metadata is standardized. Minor formatting adjustments are included.

Changes

Build and Development Tooling Configuration

Layer / File(s) Summary
Package Build Configuration
setup.py, pyproject.toml
setup.py switches from explicit package lists to find_packages() for epo_ops* discovery, adds license="MIT" metadata, updates develop extras with pyproject-fmt<3, and removes Apache license classifier. pyproject.toml build-system section reordered with same setuptools/versioningit backends.
Development Tooling Configuration
pyproject.toml, Makefile
pyproject.toml pytest configuration moved from [tool.pytest.ini_options] to [tool.pytest]; Ruff lint settings flattened from nested tables to top-level keys. Makefile format target gains pyproject-fmt --keep-full-version pyproject.toml step before ruff format.
Version Control Integration
pyproject.toml
Versioningit method switched from git to git-archive, default tag changed to v0.0.0 with v prefix, and vcs.describe-subst override added.
Code Formatting and Documentation Updates
epo_ops/models.py, tests/helpers/api_helpers.py, CHANGELOG.md
Blank-line spacing adjusted after logger initialization in models.py and around test helper function; changelog entry notes raise_for_status is now configurable.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes


Poem

🐰 A rabbit hops through config files with glee,
find_packages now bloom, pyproject.toml runs free,
Versioningit tags with a v, bold and bright,
The toolchain is cleaner—formatting feels right!

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (1 warning, 2 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'Maintenance 2026-05' is vague and does not clearly summarize the main changes; it uses a generic term without describing the actual modifications. Use a more descriptive title that highlights the primary change, such as 'Refactor: Update versioningit config and modernize setup.py' or similar.
Description check ❓ Inconclusive The description 'This and that.' is too vague and generic; it does not convey meaningful information about the changeset despite being related to maintenance work. Provide a more detailed description that explains the key changes, such as configuration updates, dependency changes, or code modernization efforts.
✅ Passed checks (2 passed)
Check name Status Explanation
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 this-and-that-202605

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
pyproject.toml (1)

94-97: ⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Add .gitattributes with export-subst for git-archive versioning.

The vcs.method = "git-archive" configuration with vcs.describe-subst = "$Format:%(describe:tags,match=v*)$" requires a .gitattributes file at the repository root to enable Git export-subst substitution. Without it, archive builds will fall back to vcs.default-tag = "v0.0.0" instead of detecting the actual version.

Add .gitattributes with:

pyproject.toml export-subst
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@pyproject.toml` around lines 94 - 97, The pyproject.toml uses vcs.method =
"git-archive" and vcs.describe-subst = "$Format:%(describe:tags,match=v*)$" but
there is no .gitattributes to enable export-subst, so archive builds will fall
back to vcs.default-tag; add a .gitattributes file at the repo root containing
the line "pyproject.toml export-subst", commit it, and push so git-archive can
substitute the describe value into pyproject.toml during archive builds.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@pyproject.toml`:
- Around line 94-97: The pyproject.toml uses vcs.method = "git-archive" and
vcs.describe-subst = "$Format:%(describe:tags,match=v*)$" but there is no
.gitattributes to enable export-subst, so archive builds will fall back to
vcs.default-tag; add a .gitattributes file at the repo root containing the line
"pyproject.toml export-subst", commit it, and push so git-archive can substitute
the describe value into pyproject.toml during archive builds.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f3317362-aa58-4eb8-9007-d46a37e1caad

📥 Commits

Reviewing files that changed from the base of the PR and between 033fcc0 and 908dad0.

📒 Files selected for processing (6)
  • CHANGELOG.md
  • Makefile
  • epo_ops/models.py
  • pyproject.toml
  • setup.py
  • tests/helpers/api_helpers.py

@codecov
Copy link
Copy Markdown

codecov Bot commented May 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.54%. Comparing base (033fcc0) to head (073a771).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #161   +/-   ##
=======================================
  Coverage   99.54%   99.54%           
=======================================
  Files          18       18           
  Lines         442      442           
=======================================
  Hits          440      440           
  Misses          2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@amotl amotl merged commit 4f839d0 into main May 16, 2026
6 checks passed
@amotl amotl deleted the this-and-that-202605 branch May 16, 2026 22:51
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