Skip to content

Conversation

@youtux
Copy link
Contributor

@youtux youtux commented Jun 28, 2025

No description provided.

@youtux youtux requested a review from Copilot June 28, 2025 03:57
Copy link

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 enhances the CI pipeline by adding a new build job that installs necessary build tools, builds distribution artifacts, and uploads them for use in automated testing and PyPI publishing.

  • Updates the releasing process documentation in RELEASING.md to include explicit commit and tagging steps.
  • Introduces a new GitHub Actions workflow job for building the package and handling artifacts, and updates the test-run and pypi-publish jobs accordingly.

Reviewed Changes

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

File Description
RELEASING.md Updated release instructions to commit changes and push a tag.
.github/workflows/main.yml Added build job, updated checkout and setup actions, and integrated artifact handling for testing and PyPI publishing.
Comments suppressed due to low confidence (2)

.github/workflows/main.yml:110

  • The comment indicates the use of 'dist/.tar.gz' while the actual command installs 'dist/.whl'. Please update the comment to accurately reflect the file being installed.
          # Using `installpkg dist/*.tar.gz` because we want to install the pre-built package (want to test against that)

.github/workflows/main.yml:120

  • Changing 'fail_ci_if_error' from true to false means Codecov errors will not fail the CI. Please confirm that this behavior is intentional to avoid potential issues with code quality checks.
        fail_ci_if_error: false

@codecov
Copy link

codecov bot commented Jun 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.76%. Comparing base (9999a2f) to head (0d501fb).
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #240   +/-   ##
=======================================
  Coverage   97.76%   97.76%           
=======================================
  Files          14       14           
  Lines         852      852           
  Branches       51       51           
=======================================
  Hits          833      833           
  Misses          8        8           
  Partials       11       11           

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

@youtux youtux merged commit 93b1036 into master Jun 28, 2025
14 of 15 checks passed
@youtux youtux deleted the ab/add-automated-release branch June 28, 2025 04:00
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