deploy-pages
deploy-pages copied to clipboard
Question: Deployment error instead of skipped when triggered from pull-request
Hi there,
Just try out this action to deploy our website, I noticed that if the workflow is triggered by pull-request it's error without any info, like this
![Screen Shot 2022-08-01 at 05 24 08](https://user-images.githubusercontent.com/508665/182047675-f14f2d83-c601-4661-a057-3cb9b77ad566.png)
while in the documentation stated as it should be skipped
This step is skipped if the workflow was triggered by a pull request.
Is it a bug or actually an expected feature?
Regards,
Apparently it still needs to have if: github.event_name == 'push'
to make it skipped on pull-request.
deploy:
runs-on: ubuntu-latest
if: github.event_name == 'push'
needs: build
![Screen Shot 2022-08-01 at 05 43 23](https://user-images.githubusercontent.com/508665/182048270-cdc9bc66-e429-44ec-b2b6-959f6a00e4a9.png)
@feryardiant Thank you for opening this. We will correct the documentation. We have decided to have the starter workflows only trigger on the default branch and not track pull requests at the moment (e.g. here).
Having it running from pull-request would be awesome as it would enable to host previews of the site changes in pages directly.
(ideally in a subfolder like https://user.github.io/project/pull-request/123/
)
Well I am late but the documentation change was merged a while ago!
@NicolasMassart One day!