Skip to content

Conversation

Copy link

Copilot AI commented Jan 22, 2026

Addresses review comments from merks and HeikoKlare on draft PR guidelines in CONTRIBUTING.md.

Changes

  • Markdown formatting: New sentences within bullet points now start on new lines with proper indentation (2 spaces), improving readability and following markdown conventions
  • Style cleanup: Removed unnecessary italics from e.g.
  • Enhanced guidance: Added note that PR creators should explicitly state what feedback they seek in the description, since draft PRs may be overlooked otherwise

Example

Before:

- A shield against comments or reviews. Marking a PR as draft does not mean "leave me alone" – it means you welcome early feedback and collaboration.

After:

- A shield against comments or reviews.
  Marking a PR as draft does not mean "leave me alone" – it means you welcome early feedback and collaboration.

The feedback guidance addition:

- You are **seeking early feedback** from the community on your approach, design decisions, or implementation.
  **Important:** Clearly state in the PR description what kind of feedback you are seeking, as draft PRs may be overlooked until marked as ready for review.
Original prompt

Please address review comment from merks and HeikoKlare here eclipse-platform#260


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Format bullets with new sentences on new lines
- Remove italics from e.g.
- Add guidance about specifying desired feedback in PR description

Co-authored-by: laeubi <1331477+laeubi@users.noreply.github.com>
Copilot AI changed the title [WIP] Address review comments from merks and HeikoKlare Address review feedback on draft PR guidelines formatting and content Jan 22, 2026
Copilot AI requested a review from laeubi January 22, 2026 11:59
@laeubi laeubi marked this pull request as ready for review January 22, 2026 12:06
@laeubi laeubi merged commit 015a84b into draft_prs Jan 22, 2026
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.

2 participants