Skip to content

Commit 38b3696

Browse files
committed
polish: improvements
1 parent 422370c commit 38b3696

File tree

1 file changed

+112
-57
lines changed

1 file changed

+112
-57
lines changed

.github/pull_request_template.md

Lines changed: 112 additions & 57 deletions
Original file line numberDiff line numberDiff line change
@@ -1,93 +1,148 @@
1+
## 📝 Before you begin
2+
3+
By opening this PR, you agree to the terms of the [Auth0 Code of Conduct](https://github.com/auth0/open-source-template/blob/master/CODE-OF-CONDUCT.md). For guidance on creating a high-quality PR, see the [Contributing Guidelines](CONTRIBUTING.md).
4+
5+
---
6+
17
## ✏️ Changes
28

3-
Please describe what this PR changes and why. If this touches multiple documentation sites (`main/` and `auth4genai/`), please call that out.
9+
Describe what this PR changes and why. Keep it brief.
10+
11+
- What problem does it solve?
12+
- Which docs or UI areas are affected?
13+
- Does it span more than one docs site (`main/`, `auth4genai/`) or the shared UI library (`ui/`)?
414

515
## 📁 Documentation scope
616

7-
**Site(s) affected:**
8-
- [ ] `main/` - Auth0 main documentation
9-
- [ ] `auth4genai/` - Auth0 for AI Agents documentation
10-
- [ ] `ui/` - Shared component library
11-
- [ ] Infrastructure/tooling
17+
**Docs site(s) affected**
18+
19+
- [ ] `main/`
20+
- [ ] `auth4genai/`
21+
22+
**Other areas**
23+
24+
- [ ] Shared UI library (`ui/`)
25+
- [ ] Infrastructure or tooling only
1226

13-
**Type of change:**
14-
- [ ] New documentation
15-
- [ ] Update to existing documentation
16-
- [ ] Navigation/information architecture change
27+
**Type of change**
28+
29+
- [ ] New page or major section
30+
- [ ] Update to existing content
31+
- [ ] Navigation or structure change
1732
- [ ] Code examples or snippets
1833
- [ ] UI component or Mintlify configuration
19-
- [ ] Linting/tooling/workflows
34+
- [ ] Tooling, workflows, or linting
35+
36+
Add paths or permalinks if it helps reviewers.
2037

2138
## 🔗 References
2239

23-
Link to relevant GitHub issues, specifications, or related public discussions.
40+
Link any related issues, design docs, or connected PRs.
41+
42+
- Issue(s): #
43+
- Related work:
2444

2545
## 🎯 Testing
2646

27-
**Local validation:**
28-
- [ ] Ran `mint dev` in affected docs folder(s)
29-
- [ ] Checked for broken links with `mint broken-links` (if applicable)
30-
- [ ] Verified accessibility with `mint a11y` (if applicable)
31-
- [ ] Built UI library with `npm run build` (if UI changes)
47+
Mark what you validated and note anything skipped.
48+
(Unit tests do not apply in this repo.)
49+
50+
**Local validation**
51+
52+
- [ ] Content-only change (no build or navigation impact)
53+
- [ ] `mint dev` in affected folders
54+
- [ ] `mint broken-links` (when applicable)
55+
- [ ] `mint a11y` (when applicable)
56+
- [ ] `npm run build` in `ui/` (for UI changes)
57+
58+
**Style and structure**
3259

33-
**Style compliance:**
34-
- [ ] Follows the Contribution Guidelines (TBD)
35-
- [ ] Uses correct Auth0 terminology and sentence case for headings
36-
- [ ] Code blocks include language and filename where appropriate
60+
- [ ] Follows the Contributing Guidelines
61+
- [ ] Uses correct Auth0 terminology
62+
- [ ] Code blocks include language and filenames when needed
63+
64+
**Repo checks**
65+
66+
- [ ] All required GitHub checks are passing
67+
- [ ] The correct base branch is used
68+
69+
If something was not run or does not apply, note it here.
3770

3871
## 🔄 Redirects
3972

40-
**Please add redirects if:**
41-
- [ ] Pages are being removed or URLs are changing
42-
- [ ] Navigation structure changes affect existing URLs
73+
Use this section only if URLs change.
74+
75+
- [ ] Pages removed or permalinks changed
76+
- [ ] Navigation changes that alter existing URLs
77+
78+
If redirects are needed, update the relevant `docs.json` and list them here:
4379

44-
If redirects are needed, please add them to the `redirects` array in the appropriate `docs.json` file:
4580
```json
46-
{
47-
"source": "/docs/old-path",
48-
"destination": "/docs/new-path"
49-
}
50-
```
81+
[
82+
{
83+
"source": "/docs/old-path",
84+
"destination": "/docs/new-path"
85+
}
86+
]
87+
````
88+
89+
## 🌍 Internationalization (main docs only)
90+
91+
* [ ] Not applicable - no English content changes in `main/`
92+
* [ ] English only; localization handled separately
93+
* [ ] Includes updates to localized content (for example, `main/docs/fr-ca/`, `main/docs/ja-jp/`)
5194

52-
## 🌍 Internationalization
95+
Add notes if something needs follow-up in localized folders.
5396

54-
**Localized content (main docs only):**
55-
- [ ] Not applicable - no English content changes
56-
- [ ] Changes apply to English content only (localized versions will be updated separately)
57-
- [ ] Changes include updates to localized versions (`fr-ca/`, `ja-jp/`)
97+
## 🧩 Impact and dependencies
5898

59-
**Note:** Main docs support French Canadian (`main/docs/fr-ca/`) and Japanese (`main/docs/ja-jp/`). If you're updating existing pages, consider whether localized versions need corresponding updates.
99+
**User-facing impact**
60100

61-
## 📊 Impact
101+
Summarize what readers will notice:
62102

63-
**User-facing changes:**
64-
Please describe any user-facing impact:
65-
- Does this introduce new pages, remove pages, or change URLs?
66-
- Does this document a new feature, Early Access capability, or breaking change?
67-
- Are there plan restrictions (Enterprise/Professional) that need callouts?
103+
* New or removed pages
104+
* URL changes
105+
* New feature, Early Access feature, or breaking change
106+
* Plan restrictions that need callouts
68107

69-
**Dependencies:**
70-
Please note any dependencies or coordination needs:
71-
- Are there related PRs or release timing considerations?
72-
- Does this require coordination with Product, Legal, or Security teams?
108+
**Dependencies**
109+
110+
List anything this PR depends on or influences:
111+
112+
* Product release timing
113+
* Related PRs or repos
114+
* Follow-up docs needed after merge
73115

74116
## 🚀 Deployment
75117

76-
✅🚫 This can be deployed any time (Mintlify auto-deploys on merge to main)
118+
* [ ] Safe to deploy on merge
119+
* [ ] Do not merge until a condition is met
120+
121+
If blocked, state the condition:
77122

78-
<!--
79-
OR
80-
⚠️ This should not be merged until:
81-
- Other PR is merged because REASON
82-
- After date because REASON
83-
- Other condition: REASON
84-
-->
123+
* Dependent PRs:
124+
* Date or window:
125+
* Other requirement:
85126

86127
## 📝 Follow-up
87128

88-
- [ ] No follow-up needed
89-
- [ ] Follow-up issue/PR needed (describe below)
129+
* [ ] No follow-up needed
130+
* [ ] Follow-up work required
131+
132+
Add links or describe next steps if needed.
90133

91134
## 💬 Additional context
92135

93-
Please add any other context, screenshots, or preview links that would help reviewers.
136+
Optional. Add anything reviewers should know:
137+
138+
* Specific pages in the Mintlify preview you want checked
139+
* Screenshots or diagrams (with alt text)
140+
* Notes for specific reviewers
141+
142+
> [!TIP]
143+
> Useful references:
144+
>
145+
> * [Contributing Guidelines](CONTRIBUTING.md)
146+
> * Mintlify documentation: [https://www.mintlify.com/docs](https://www.mintlify.com/docs)
147+
> * [main README](main/README.md)
148+
> * [auth4genai README](auth4genai/README.md)

0 commit comments

Comments
 (0)