Skip to content
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

Add github actions yml #16

Merged
merged 26 commits into from
Feb 8, 2021
Merged

Conversation

Test-Automation-Engineer
Copy link
Contributor

No description provided.

@mjcarsjens
Copy link
Contributor

mjcarsjens commented Jan 28, 2021

Ziet er goed uit! Even als check of mijn beeldvorming klopt:

Bij pushes naar zowel development als master en feature branches wordt Storybook gebouwd om te kijken of dit nog goed gaat, en bij development en master wordt deze build vervolgens naar respectievelijk de staging of de productie storybook omgeving gepusht.

Er valt mij wel 1 ding op, hieronder (bij de laatste commit) staat dat de build 2x is getriggered, 1x voor push en 1x voor pull request (naar mijn vermoeden omdat er eerst een push is gedaan naar de feature-branch, en vervolgens is die commit gebruikt om een pull request aan te maken naar development). Opzich geen groot probleem, want je wilt hem zeker triggeren bij pull requests naar development en naar master, en builden op feature branch commits is in mijn ogen ook zeker mooi meegenomen, maar het kijkt gewoon een beetje gek zo.

@Test-Automation-Engineer
Copy link
Contributor Author

Test-Automation-Engineer commented Jan 28, 2021

Ziet er goed uit! Even als check of mijn beeldvorming klopt:

Bij pushes naar zowel development als master en feature branches wordt Storybook gebouwd om te kijken of dit nog goed gaat, en bij development en master wordt deze build vervolgens naar respectievelijk de staging of de productie storybook omgeving gepusht.

Er valt mij wel 1 ding op, hieronder (bij de laatste commit) staat dat de build 2x is getriggered, 1x voor push en 1x voor pull request (naar mijn vermoeden omdat er eerst een push is gedaan naar de feature-branch, en vervolgens is die commit gebruikt om een pull request aan te maken naar development). Opzich geen groot probleem, want je wilt hem zeker triggeren bij pull requests naar development en naar master, en builden op feature branch commits is in mijn ogen ook zeker mooi meegenomen, maar het kijkt gewoon een beetje gek zo.

Yes, only for development and master branch Storybook will be deployed to Azure.

Copy link
Contributor

@mjcarsjens mjcarsjens left a comment

Choose a reason for hiding this comment

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

Ouch, I commented in dutch on my previous comment, my bad.. But thanks for the clarification, we can merge this PR now 😄

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.

3 participants