Skip to content

Conversation

@mulkieran
Copy link
Member

@mulkieran mulkieran commented Oct 21, 2025

Summary by CodeRabbit

  • Chores
    • Updated development environment container images to Fedora 42.
    • Added libatomic to lint/analysis dependencies in CI checks.
    • Established Python 3.12 as the minimum required version for installation.

@mulkieran mulkieran self-assigned this Oct 21, 2025
@mulkieran mulkieran moved this to In Progress in 2025October Oct 21, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 21, 2025

Walkthrough

Updated GitHub Actions workflow container images from fedora:41 to fedora:42 and added libatomic to the development_environment_checks matrix; setup.cfg now requires Python >=3.12. No workflow control flow or steps were changed.

Changes

Cohort / File(s) Summary
GitHub Actions — main workflow
.github/workflows/main.yml
Container image updated from fedora:41 to fedora:42; added libatomic to the development_environment_checks matrix dependencies.
GitHub Actions — push workflow
.github/workflows/push.yml
Container image updated from fedora:41 to fedora:42. No other step or control-flow changes.
Packaging / Metadata
setup.cfg
Added python_requires = >=3.12 under [options] to enforce minimum Python version.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped from forty-one to forty-two today,
Added libatomic along the way,
Python twelve now leads the trail,
A tidy bump — a little tail-wag and a happy trail! 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Title Check ✅ Passed The pull request title "Increase CURRENT DEVELOPMENT ENVIRONMENT to Fedora 42" directly and accurately describes the primary change in the changeset: upgrading the container images from Fedora 41 to Fedora 42 across multiple workflow files (main.yml and push.yml). The title is specific, clear, and concise, allowing a teammate reviewing history to immediately understand the main objective. While the PR includes secondary changes such as adding libatomic to dependencies and setting python_requires in setup.cfg, the title appropriately focuses on the most prominent change without requiring exhaustive coverage of all modifications, which aligns with the principle that titles need not detail every aspect of the changeset.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
@mulkieran mulkieran changed the title Issue project 805 Increase CURRENT DEVELOPMENT ENVIRONMENT to Fedora 42 Oct 21, 2025
@mulkieran mulkieran merged commit f0fb69f into stratis-storage:master Oct 21, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in 2025October Oct 21, 2025
@mulkieran mulkieran moved this from Done to Done(3) in 2025October Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done(3)

Development

Successfully merging this pull request may close these issues.

1 participant