-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Intro to open source with knative blog series p0 and p1 #5618
Conversation
Signed-off-by: Calum Murray <cmurray@redhat.com>
✅ Deploy Preview for knative ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
Co-authored-by: Leo Li <leoli@redhat.com>
Signed-off-by: Calum Murray <cmurray@redhat.com>
/hold until @Leo6Leo adds images, but the content itself is ready for review |
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.
The first round of review!
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
/unhold |
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.
Great to have such a series :+1 Thanks a lot for working on this!
Left a few comments
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.
LGTM. Thank you for this 🎉
Signed-off-by: Calum Murray <cmurray@redhat.com> Co-authored-by: Christoph Stäbler <cstabler@redhat.com>
@creydr I think i have addressed all your comments, could you review when you have time? |
Signed-off-by: Calum Murray <cmurray@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
Co-authored-by: Leo Li <leoli@redhat.com>
/lgtm |
/approve |
/hold unhold once ready |
Co-authored-by: Paul Schweigert <paul@paulschweigert.com>
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Cali0707, pierDipi, psschwei The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/unhold |
This PR contains a draft of the first post in the blog series we proposed recently to make it easier for new contributors to contribute code. The proposal can be seen here
Proposed Changes
We want to publish the first two posts together, so this PR will be in draft status until the second post is also ready