Skip to content

Commit b113dba

Browse files
committed
polish: improvements
1 parent 422370c commit b113dba

File tree

1 file changed

+110
-57
lines changed

1 file changed

+110
-57
lines changed

.github/pull_request_template.md

Lines changed: 110 additions & 57 deletions
Original file line numberDiff line numberDiff line change
@@ -1,93 +1,146 @@
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+
15
## ✏️ Changes
26

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

513
## 📁 Documentation scope
614

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

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

2136
## 🔗 References
2237

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

2543
## 🎯 Testing
2644

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)
45+
Mark what you validated and note anything skipped.
46+
(Unit tests do not apply in this repo.)
47+
48+
**Local validation**
49+
50+
- [ ] Content-only change (no build or navigation impact)
51+
- [ ] `mint dev` in affected folders
52+
- [ ] `mint broken-links` (when applicable)
53+
- [ ] `mint a11y` (when applicable)
54+
- [ ] `npm run build` in `ui/` (for UI changes)
55+
56+
**Style and structure**
57+
58+
- [ ] Follows the Contributing Guidelines
59+
- [ ] Uses correct Auth0 terminology
60+
- [ ] Code blocks include language and filenames when needed
3261

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
62+
**Repo checks**
63+
64+
- [ ] All required GitHub checks are passing
65+
- [ ] The correct base branch is used
66+
67+
If something was not run or does not apply, note it here.
3768

3869
## 🔄 Redirects
3970

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

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

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

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/`)
95+
## 🧩 Impact and dependencies
5896

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.
97+
**User-facing impact**
6098

61-
## 📊 Impact
99+
Summarize what readers will notice:
62100

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?
101+
* New or removed pages
102+
* URL changes
103+
* New feature, Early Access feature, or breaking change
104+
* Plan restrictions that need callouts
68105

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?
106+
**Dependencies**
107+
108+
List anything this PR depends on or influences:
109+
110+
* Product release timing
111+
* Related PRs or repos
112+
* Follow-up docs needed after merge
73113

74114
## 🚀 Deployment
75115

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

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-
-->
121+
* Dependent PRs:
122+
* Date or window:
123+
* Other requirement:
85124

86125
## 📝 Follow-up
87126

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

91132
## 💬 Additional context
92133

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

0 commit comments

Comments
 (0)