Skip to content

Conversation

@LFDanLu
Copy link
Member

@LFDanLu LFDanLu commented Sep 24, 2019

Instead of triggering on every push event, we can trigger on pull request + pull request update (new commits to a pull). This should lower the number of builds and number of errors that occur when the test build tries to comment on a non-existent PR.

If we want a storybook for manual testing before a formal release, we can get one from opening a pull from master to next.

See https://help.github.com/en/articles/events-that-trigger-workflows#pull-request-event-pull_request

@LFDanLu LFDanLu changed the title Testing pull_request trigger Changing test trigger to pull_requests only Sep 24, 2019
@github-actions
Copy link

Build successful! View the storybook

@github-actions
Copy link

Build successful! View the storybook

@LFDanLu LFDanLu closed this Sep 26, 2019
@LFDanLu
Copy link
Member Author

LFDanLu commented Sep 26, 2019

Closing since we don't want this behavior actually. I'll fix the no-pr build check in a separate PR

@LFDanLu LFDanLu deleted the test_pulls_only branch September 26, 2019 22:37
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.

2 participants