Skip to content

ci: using trusted publisher release process#3171

Merged
RobPasMue merged 7 commits intomainfrom
maint/trusted_publisher_release
Jun 18, 2024
Merged

ci: using trusted publisher release process#3171
RobPasMue merged 7 commits intomainfrom
maint/trusted_publisher_release

Conversation

@clatapie
Copy link
Contributor

Following PyAnsys guidelines, this PR is to implement the ansys/actions/release-pypi-public.

@clatapie clatapie self-assigned this Jun 17, 2024
@clatapie clatapie requested a review from a team as a code owner June 17, 2024 13:19
@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc Maintenance labels Jun 17, 2024
@codecov
Copy link

codecov bot commented Jun 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.93%. Comparing base (7c1eb1e) to head (7458a59).
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3171      +/-   ##
==========================================
- Coverage   86.63%   83.93%   -2.71%     
==========================================
  Files          52       52              
  Lines        9550     9550              
==========================================
- Hits         8274     8016     -258     
- Misses       1276     1534     +258     

Copy link
Collaborator

@germa89 germa89 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you a lot for this @clatapie.

Can you perform a patch release after this? After all the changes you did it might be a good idea.

I wonder if we should add the other PRs made before towncrier PR to the changelog manually. What is the recommended approach?

@clatapie clatapie requested a review from RobPasMue June 18, 2024 10:18
Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

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

LGTM! Removing PYPI_TOKEN

@RobPasMue RobPasMue changed the title maint: using trusted publisher release process ci: using trusted publisher release process Jun 18, 2024
@RobPasMue
Copy link
Member

@clatapie - be mindful of the commit name. "maint" is not allowed.

@clatapie
Copy link
Contributor Author

@clatapie - be mindful of the commit name. "maint" is not allowed.

Indeed, thank you @RobPasMue!

@RobPasMue RobPasMue enabled auto-merge (squash) June 18, 2024 10:30
Copy link
Contributor

@SMoraisAnsys SMoraisAnsys left a comment

Choose a reason for hiding this comment

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

LGTM

@RobPasMue RobPasMue merged commit 8887d07 into main Jun 18, 2024
@RobPasMue RobPasMue deleted the maint/trusted_publisher_release branch June 18, 2024 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Related with CICD, Github Actions, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants