-
Notifications
You must be signed in to change notification settings - Fork 69
📖Docs: remove duplicate Open a Pull Request section from CONTRIBUTING.md #869
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
📖Docs: remove duplicate Open a Pull Request section from CONTRIBUTING.md #869
Conversation
Signed-off-by: JANHVI BABANI <114232474+Janhvibabani@users.noreply.github.com>
✅ Deploy Preview for kubestellar-docs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Hi @Janhvibabani. Thanks for your PR. I'm waiting for a kubestellar member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Welcome to KubeStellar! 🚀 Thank you for submitting this Pull Request. Before your PR can be merged, please ensure: ✅ DCO Sign-off - All commits must be signed off with ✅ PR Title - Must start with an emoji: ✨ (feature), 🐛 (bug fix), 📖 (docs), 🌱 (infra/tests), Getting Started with KubeStellar: Contributor Resources:
🌟 Help KubeStellar Grow - We Need Adopters! Our roadmap is driven entirely by adopter feedback. Whether you're using KubeStellar yourself or know someone who could benefit from multi-cluster Kubernetes: 📋 Take our Multi-Cluster Survey - Share your use cases and help shape our direction! A maintainer will review your PR soon. Feel free to ask questions in the comments or on Slack! |
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.
Pull request overview
Removes duplicated contribution instructions in CONTRIBUTING.md to eliminate redundant guidance and improve readability (Fixes #868).
Changes:
- Deleted the duplicated “### 8. Open a Pull Request” section from
CONTRIBUTING.md.
Comments suppressed due to low confidence (1)
CONTRIBUTING.md:90
- There is still a duplicate "## Contribution Guidelines" section (the same content appears again starting around line 106). Consider removing one of the duplicated blocks so the document doesn’t repeat itself in back-to-back sections.
## Contribution Guidelines
- **Write Clearly:** Use concise language and proper formatting.
- **Stay Consistent:** Maintain the existing structure and style.
- **Respect Internationalization Standards:** Avoid pushing raw UI strings directly; always use i18n references.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/lgtm |
|
LGTM label has been added. DetailsGit tree hash: 2da8f132cafe296783c1dcec78b5a11480355ec2 |
|
/ok-to-test |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: oksaumya The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
e667b54
into
kubestellar:main
…md (kubestellar#869) Signed-off-by: JANHVI BABANI <114232474+Janhvibabani@users.noreply.github.com>
📌 Fixes
Fixes #868
📝 Summary of Changes
remove duplicate “Open a Pull Request” section from CONTRIBUTING.md
Changes Made
Checklist
Please ensure the following before submitting your PR:
Screenshots or Logs (if applicable)
👀 Reviewer Notes
Add any special notes for the reviewer here