Skip to content

[internal] Update pull request template for clarity and consistency#269

Open
dozro wants to merge 2 commits intoSableClient:devfrom
dozro:internal-change/adjust-pr-issue-templates
Open

[internal] Update pull request template for clarity and consistency#269
dozro wants to merge 2 commits intoSableClient:devfrom
dozro:internal-change/adjust-pr-issue-templates

Conversation

@dozro
Copy link
Contributor

@dozro dozro commented Mar 14, 2026

Description

This pull request updates the pull request template to align with the new project repository, clarify instructions, and introduce a section about generative AI usage. The changes help ensure contributors follow updated guidelines and document AI involvement in code contributions.

Template updates and clarifications:

  • Updated repository and contribution guideline links to reference SableClient/Sable instead of the previous project.
  • Improved the language and clarity of instructions, including the description and type of change sections.
  • Revised checklist items for consistency and added a reminder to create a required changeset for changes.

Generative AI disclosure:

  • Added a new section asking contributors to disclose whether generative AI was used in code contributions and to confirm adherence to project restrictions on AI-generated code.

@dozro dozro requested a review from a team March 14, 2026 17:44
@7w1 7w1 added the internal label Mar 14, 2026
@hazre
Copy link
Collaborator

hazre commented Mar 14, 2026

Do we really need the AI disclosure part? the whole template is kinda useless as is, we don't need more stuff I feel like.

Removed the section regarding generative AI usage from the pull request template.
@hazre
Copy link
Collaborator

hazre commented Mar 14, 2026

frankly I think we should get rid of the whole thing. or make it more actually useful. @7w1 @Just-Insane any thoughts?

@7w1
Copy link
Member

7w1 commented Mar 14, 2026

I don’t mind getting rid of it or at least the checkbox part, none of that feels particularly helpful.

@dozro
Copy link
Contributor Author

dozro commented Mar 14, 2026

fine by me, just thought having a link to cinny's repo wouldn't be useful 🤷‍♀️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants