-
Notifications
You must be signed in to change notification settings - Fork 984
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
docs: filter to main for example GH Pages config #1715
Conversation
@@ -43,6 +43,7 @@ name: Build and deploy GH Pages | |||
jobs: | |||
build: | |||
runs-on: ubuntu-latest | |||
if: github.ref == 'refs/heads/main' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we just point out that this branch name might change depending on the person in the docs below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry about the delayed response! Let me know if the second commit is sufficient 🙂
Oops, that third commit serves me right for not checking 😅 Happy to squash the commits if that makes your life easier! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's fine, thanks!
* docs: filter to main for example GH Pages config * docs: update GH Pages wording to indicate choice of branch * docs: typo fix
* docs: filter to main for example GH Pages config * docs: update GH Pages wording to indicate choice of branch * docs: typo fix
Sanity check:
This just bit me - the example configuration for automatically building and deploying your website on GitHub Pages doesn't filter for the main branch, so my draft branch ended up being deployed...
Pretty simple fix, shouldn't cause any issues 🙂