Enhance first-time contributor welcome message#6439
Merged
Conversation
Updated the welcome message for first-time contributors to include additional information and resources. Signed-off-by: Ayush More <ayushmore42595@gmail.com>
Contributor
Author
eeshaanSA
requested changes
Jan 16, 2026
Comment on lines
+61
to
+62
|
|
||
| This is required by the Developer Certificate of Origin (DCO). |
Contributor
There was a problem hiding this comment.
This sentence doesn't make sense. DCO is a thing, it doesn't 'require' anything.
| This is required by the Developer Certificate of Origin (DCO). | ||
| --- | ||
| ### Run checks locally | ||
| Before pushing updates, please run: |
Contributor
There was a problem hiding this comment.
Better grammar and readability needed.
| --- | ||
| If anything is unclear, feel free to ask in this PR or on Slack. We’re happy to help | ||
|
|
||
| Thanks for contributing to PipeCD! |
Contributor
There was a problem hiding this comment.
Should we also include the link to join our channel?
eeshaanSA
requested changes
Jan 16, 2026
Updated the first-time contributor message to improve clarity and added helpful resources. Signed-off-by: Ayush More <ayushmore42595@gmail.com>
eeshaanSA
requested changes
Jan 19, 2026
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
eeshaanSA
requested changes
Jan 19, 2026
|
|
||
| ### 💬 Need help? | ||
| If anything is unclear, feel free to ask in this PR or join us on Slack: | ||
| https://communityinviter.com/apps/cloud-native/cncf (channel: \`#pipecd\`) |
Contributor
There was a problem hiding this comment.
why do we need backticks? Infact, why do we need the channel name at all? Just put a note saying join the #pipecd channel on the CNCF Slack. Get your invite
Thats better in my opinion.
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
eeshaanSA
requested changes
Jan 19, 2026
Signed-off-by: Eeshaan Sawant <100678386+eeshaanSA@users.noreply.github.com>
eeshaanSA
approved these changes
Jan 19, 2026
Contributor
eeshaanSA
left a comment
eeshaanSA
left a comment
There was a problem hiding this comment.
LGTM, I dont know how this will turn out. We can only find out if once merged. Maybe you can test it on a local fork and give us some screenshots?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated the welcome message for first-time contributors to include additional information and resources.
What this PR does:
Why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: