-
Notifications
You must be signed in to change notification settings - Fork 0
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
(baby-project-deployment) #3 Creates a simple deployment action #5
Conversation
22273e7
to
88e7cfe
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.
I left two minor comments about the README that might be good to change, other than that it looks good!
I don't have a whole lot of experience with CI actions so it might be good to wait from someone else to Approve it as well.
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 work! 👍
I left a couple of comments but in general, I think it looks really solid! 👊
Awesome points you guys @OttoFI and @AndreasNasman thanks for the review. I'll fix these later today and request a new reivew 👍🏼 |
Co-authored-by: Otto Lindén <[email protected]>
Co-authored-by: Otto Lindén <[email protected]>
Co-authored-by: Andreas Näsman <[email protected]>
Co-authored-by: Andreas Näsman <[email protected]>
Co-authored-by: Andreas Näsman <[email protected]>
Co-authored-by: Andreas Näsman <[email protected]>
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.
Some comments from Grammarly.
Co-authored-by: Andreas Näsman <[email protected]>
Co-authored-by: Andreas Näsman <[email protected]>
Co-authored-by: Andreas Näsman <[email protected]>
Changes micro repo to poly repo because that seems to be the more used term
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 work!
I created the pipline prosess for deploying to the server. This only runs when something has been merged to main. The idea in the future is that each PR will create its own instance of the deployment. But that is something we can add in the future.
The PR deployment will be like a RC (Release Candidate). That way we can test all or latest feautres agenst production before we actualy merge it and it gets deployed to production.
But for now we only have the main branch that is essentaly production