-
Notifications
You must be signed in to change notification settings - Fork 645
Improve adr-000-template.md #5957
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
Conversation
|
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
This PR refines the project's ADR template to enhance its clarity and usability.
- Updated the title header and added a date field.
- Replaced the table in the Status section with inline guidance and added an Implementation descriptor.
- Introduced Consequences and Alternatives sections to provide additional decision context.
size-limit report 📦
|
|
@hectahertz I see the "stress tests" job is failing, although I can't imagine anything in this PR was the cause. Could you please take a look and advise? |
|
@lesliecdubs they were flaky, just submitted a fix. Feel free to ignore :) |
Changelog
Improves the project's ADR template with additional sections and better formatting, in alignment with https://github.com/github/primer/pull/5111 (GitHub staff only)
Changed
Improved content and formatting in the ADR template
Rollout strategy
Testing & Reviewing
Review the diff
Merge checklist