Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update deploy.yml #643

Merged
merged 3 commits into from
Oct 30, 2024
Merged

Update deploy.yml #643

merged 3 commits into from
Oct 30, 2024

Conversation

stephanmg
Copy link
Collaborator

Upgrade actions/checkout@v2 -> actions/checkout@v4.

Upgrade actions/checkout@v2 -> actions/checkout@v4.
@stephanmg stephanmg requested a review from arrjon October 29, 2024 15:32
Copy link
Collaborator

@arrjon arrjon left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov-commenter
Copy link

codecov-commenter commented Oct 29, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.29%. Comparing base (634efa1) to head (1014fa7).
Report is 1 commits behind head on develop.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #643      +/-   ##
===========================================
- Coverage    84.31%   84.29%   -0.03%     
===========================================
  Files          141      141              
  Lines         8544     8544              
===========================================
- Hits          7204     7202       -2     
- Misses        1340     1342       +2     

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

@stephanmg
Copy link
Collaborator Author

stephanmg commented Oct 29, 2024

Associated with the deploy.yml - API keys need to re-generated, otherwise deploy GHA will fail.

At least this seems the most likely reason:

2024-10-29T15:30:51.7008965Z          Invalid or non-existent authentication information. See                
2024-10-29T15:30:51.7010374Z          https://pypi.org/help/#invalid-auth for more information.  

@arrjon
Copy link
Collaborator

arrjon commented Oct 29, 2024

Associated with the deploy.yml - API keys need to re-generated, otherwise deploy GHA will fail.

@yannikschaelte could you give us access?

According to docs username must be __token__ when using API token: https://pypi.org/help/#invalid-auth
We need to generate PYP_API_TOKEN once we have access. 
__token__ is required, since we do authenticate with an API token and not username password combination.
@stephanmg
Copy link
Collaborator Author

@arrjon check please

@stephanmg stephanmg merged commit 1d75532 into develop Oct 30, 2024
10 checks passed
@stephanmg stephanmg deleted the stephanmg-patch-1 branch October 30, 2024 09:05
stephanmg added a commit that referenced this pull request Oct 30, 2024
* Update deploy.yml

* Update deploy.yml (#643)

* Update deploy.yml

Upgrade actions/checkout@v2 -> actions/checkout@v4.

* Update deploy.yml

According to docs username must be __token__ when using API token: https://pypi.org/help/#invalid-auth

* Update deploy.yml

We need to generate PYP_API_TOKEN once we have access. 
__token__ is required, since we do authenticate with an API token and not username password combination.

---------

Co-authored-by: Jonas Arruda <[email protected]>
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.

3 participants