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
chore(docs): compare PRs against next branch instead of current #4904
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
- name: Create Docs Report on current | ||
if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/current' }} | ||
- name: Create Docs Report on current or next | ||
if: ${{ github.event_name == 'push' && (github.ref == 'refs/heads/current' || github.ref == 'refs/heads/next') }} |
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.
Not sure if the event_name
is still push when a PR is merged. Does anyone know?
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.
Seems fair to assume so, esp. since there doesn't seem to be a merge
event.
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.
It might be trial and error. I will merge and check
Component Testing Report Updated Sep 7, 2023 7:56 PM (UTC)
|
No changes to documentation |
- fixes token mismatch when deleting dataset - better message when no changes to documentation
412bbff
to
99e4d45
Compare
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.
Nice!
- name: Create Docs Report on current | ||
if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/current' }} | ||
- name: Create Docs Report on current or next | ||
if: ${{ github.event_name == 'push' && (github.ref == 'refs/heads/current' || github.ref == 'refs/heads/next') }} |
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.
Seems fair to assume so, esp. since there doesn't seem to be a merge
event.
oh just realized the |
Description
What to review
The actions conditionals are valid
Notes for release
N/A