Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add doc deployment job#172

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
dstansby merged 4 commits intomainfromsam-not-fork/doc-deployment-job
Jun 15, 2023
Merged

Conversation

samcunliffe
Copy link
Collaborator

@samcunliffesamcunliffe commentedJun 15, 2023
edited
Loading

This is a cherry-pick of the first commit from#166 but where the branch is not from a fork, but from@matplotlib's repo directly.

Probably you need to set up the pages from thegh-pages branch in the repo settings? Because there's nothing athttps://matplotlib.github.io/napari-matplotlib

@samcunliffesamcunliffe changed the titleSam's WIP Add doc deployment jobAdd doc deployment jobJun 15, 2023
@samcunliffesamcunliffe marked this pull request as ready for reviewJune 15, 2023 11:19
name: docs

- name: Push to GitHub pages
uses: peaceiris/actions-gh-pages@v3
Copy link
CollaboratorAuthor

Choose a reason for hiding this comment

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

Probs only want to publish the docs for commits to main.

Suggested change
uses:peaceiris/actions-gh-pages@v3
uses:peaceiris/actions-gh-pages@v3
if:github.event_name == 'push' && github.ref_name == 'main'

And we could switch to JamesIves' action.

@dstansbydstansby mentioned this pull requestJun 15, 2023
@dstansby
Copy link
Member

Upload is now working. The settings page says the site should be athttps://matplotlib.org/napari-matplotlib/, but it is not. I suggest we 🚢 this, and I'll chat to the Matplotlib org admins about how to sort out the domain stuff.

Screenshot 2023-06-15 at 13 52 51

samcunliffe reacted with rocket emoji

@dstansbydstansby merged commitbe9466c intomainJun 15, 2023
@dstansbydstansby deleted the sam-not-fork/doc-deployment-job branchJune 15, 2023 12:55
@dstansbydstansby mentioned this pull requestJun 15, 2023
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@dstansbydstansbydstansby approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@samcunliffe@dstansby

[8]ページ先頭

©2009-2025 Movatter.jp