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

Feature: Update publish trigger for docs-site #1354

Merged
merged 5 commits into from
Jan 5, 2024

Conversation

Magnus167
Copy link
Member

This PR changes the trigger event for publishing the docs on GitHub pages.
Currently, the docs are updated on push to main.
Given that the release notes on the docs-site require a release to be published on GitHub to generate notes for the latest version, the docs will now be pushed as part of the upload to PyPI (which itself is triggered when a release is published).
The old workflow for publishing docs still remains, but only with a workflow dispatch trigger.

@Magnus167 Magnus167 requested a review from a team as a code owner January 2, 2024 15:02
Copy link

codecov bot commented Jan 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3e17444) 78.79% compared to head (efe51cb) 78.79%.
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1354   +/-   ##
========================================
  Coverage    78.79%   78.79%           
========================================
  Files           62       62           
  Lines         5783     5783           
========================================
  Hits          4557     4557           
  Misses        1226     1226           

Impacted file tree graph

@Magnus167 Magnus167 merged commit 914e86a into develop Jan 5, 2024
7 checks passed
@Magnus167 Magnus167 deleted the feature/update_docs_publish_event branch January 5, 2024 13:25
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.

None yet

2 participants