Skip to content
This repository has been archived by the owner on Nov 21, 2022. It is now read-only.

Update PyPI release trigger #302

Merged

Conversation

akihironitta
Copy link
Contributor

The PyPI release workflow wasn't triggered by the release event type as reported by @rohitgr7. The exact reason is still unknown, but it could be due to types: [created]. This PR updates the trigger so that it all aligns across our repositories' settings.

See the workflow file in our repo: https://github.com/Lightning-AI/lightning/blob/4750e221d1e7622bbf2d0e5b09e7136032e9c410/.github/workflows/release-pypi.yml#L8

@codecov
Copy link

codecov bot commented Nov 4, 2022

Codecov Report

Merging #302 (61be366) into master (b6142fe) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@          Coverage Diff          @@
##           master   #302   +/-   ##
=====================================
  Coverage      74%    74%           
=====================================
  Files          63     63           
  Lines        1462   1462           
=====================================
  Hits         1086   1086           
  Misses        376    376           

Copy link
Contributor

@rohitgr7 rohitgr7 left a comment

Choose a reason for hiding this comment

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

thank you

Copy link
Contributor

@carmocca carmocca left a comment

Choose a reason for hiding this comment

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

Might be unrelated, but @Borda often has to manually publish when we release PL, or utilities

@Borda
Copy link
Member

Borda commented Nov 5, 2022

manually publish when we release PL, or utilities

Utils were always automated, and PL for a while had not implemented the multi-package flow as it has been done since 1.8

@Borda Borda merged commit 593c794 into Lightning-Universe:master Nov 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants