Skip to content

Finish release action#2944

Merged
jmthomas merged 1 commit intomainfrom
release_cleanup
Mar 6, 2026
Merged

Finish release action#2944
jmthomas merged 1 commit intomainfrom
release_cleanup

Conversation

@jmthomas
Copy link
Copy Markdown
Member

@jmthomas jmthomas commented Mar 6, 2026

No description provided.

@jmthomas jmthomas requested a review from ryanmelt March 6, 2026 16:30
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 6, 2026

uses: pypa/gh-action-pypi-publish@ed0c53931b1dc9bd32cbe73a98c7f6766f8a527e
with:
user: __token__
password: ${{ secrets.PYPI_API_TOKEN }}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think this is what caused the pypi publish to fail. Since we're now doing OIDC secure publishing, it didn't like that we were also passing a user and password.

Warning: The workflow was run with the 'attestations: true' input, but an explicit password was also set, disabling Trusted Publishing. As a result, the attestations input is ignored.
Warning: Trusted Publishers allows publishing packages to PyPI from automated environments like GitHub Actions without needing to use username/password combinations or API tokens to authenticate with PyPI. Read more: https://docs.pypi.org/trusted-publishers
Warning: A new Trusted Publisher for the currently running publishing workflow can be created by accessing the following link(s) while logged-in as an owner of the package(s):
- https://pypi.org/manage/project/openc3/settings/publishing/?provider=github&owner=OpenC3&repository=cosmos&workflow_filename=release.yml

Checking openc3/python/dist/openc3-7.0.0rc3-py3-none-any.whl: PASSED
Checking openc3/python/dist/openc3-7.0.0rc3.tar.gz: PASSED
Uploading distributions to https://upload.pypi.org/legacy/
Uploading openc3-7.0.0rc3-py3-none-any.whl
WARNING  Error during upload. Retry with the --verbose option for more details. 
ERROR    HTTPError: 403 Forbidden from https://upload.pypi.org/legacy/          
         Invalid or non-existent authentication information. See                
         https://pypi.org/help/#invalid-auth for more information.              

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.52%. Comparing base (c3ea349) to head (6ee0475).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2944      +/-   ##
==========================================
+ Coverage   78.46%   78.52%   +0.05%     
==========================================
  Files         673      673              
  Lines       55324    55324              
  Branches      728      728              
==========================================
+ Hits        43412    43444      +32     
+ Misses      11834    11802      -32     
  Partials       78       78              
Flag Coverage Δ
python 79.75% <ø> (+<0.01%) ⬆️
ruby-api 81.06% <ø> (+0.62%) ⬆️
ruby-backend 82.15% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jmthomas jmthomas merged commit 0310dda into main Mar 6, 2026
50 of 52 checks passed
@jmthomas jmthomas deleted the release_cleanup branch March 6, 2026 17:15
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