Skip to content

Actually run attestation#236

Merged
CoMPaTech merged 1 commit into
mainfrom
pep740
Jul 5, 2026
Merged

Actually run attestation#236
CoMPaTech merged 1 commit into
mainfrom
pep740

Conversation

@CoMPaTech

@CoMPaTech CoMPaTech commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Updated the release version to 0.6.11.
    • Refreshed the changelog with the latest release entry.
    • Tightened release publishing so attestations are generated consistently during publishing.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. documentation Improvements or additions to documentation labels Jul 5, 2026
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

This PR removes a redundant conditional guard on the PEP 740 attestations step in the merge workflow, bumps the package version from 0.6.10 to 0.6.11 in pyproject.toml, and updates CHANGELOG.md with a combined release header and a new bullet noting pinned GitHub actions.

Changes

Release 0.6.11 workflow and metadata update

Layer / File(s) Summary
Attestation step gating simplification
.github/workflows/merge.yml
Removes the if: steps.check_package.outputs.should_publish == 'true' guard from the PEP 740 attestations step, relying on the job-level should_publish gate instead.
Version bump and changelog update
pyproject.toml, CHANGELOG.md
Bumps project.version to 0.6.11 and updates the changelog header to a combined 0.6.10 & 0.6.11 entry with a new "Pin Github actions" bullet.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

Poem

A hop, a skip, a version bend,
0.6.11 rounds the bend,
No more guard to block the way,
Attestations run and stay,
Changelog hums a tidy tune,
Pinned actions safe 'neath the moon. 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: the attestation step is now executed during publishing.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pep740

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.

@sonarqubecloud

sonarqubecloud Bot commented Jul 5, 2026

Copy link
Copy Markdown

@CoMPaTech CoMPaTech deployed to testpypi July 5, 2026 10:22 — with GitHub Actions Active

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 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.

Inline comments:
In `@CHANGELOG.md`:
- Line 10: Update the changelog entry to use the standard GitHub capitalization:
change the existing “Github” text in the changelog bullet to “GitHub” for
consistency. Locate the affected wording in the changelog entry and keep the
rest of the sentence unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d285f0de-d5f0-433f-9938-02e4488d2c6d

📥 Commits

Reviewing files that changed from the base of the PR and between 3bd0f70 and 2b6f5c8.

📒 Files selected for processing (3)
  • .github/workflows/merge.yml
  • CHANGELOG.md
  • pyproject.toml
💤 Files with no reviewable changes (1)
  • .github/workflows/merge.yml

Comment thread CHANGELOG.md
@codecov

codecov Bot commented Jul 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.29%. Comparing base (54a587d) to head (2b6f5c8).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #236   +/-   ##
=======================================
  Coverage   85.29%   85.29%           
=======================================
  Files           8        8           
  Lines         775      775           
=======================================
  Hits          661      661           
  Misses        114      114           

☔ View full report in Codecov by Harness.
📢 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.

@CoMPaTech CoMPaTech merged commit 5e8db55 into main Jul 5, 2026
14 checks passed
@CoMPaTech CoMPaTech deleted the pep740 branch July 5, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant