Skip to content

Commit 2949f40

Browse files
Copilotctauchen
andcommitted
Add requirement for Netlify deploy preview links in PR descriptions
Co-authored-by: ctauchen <39195715+ctauchen@users.noreply.github.com>
1 parent c524465 commit 2949f40

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

.github/copilot-instructions.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -244,6 +244,9 @@ yarn test:screenshots # Visual regression tests (optional)
244244
- Build must complete successfully
245245
- Tests must pass
246246
- Deploy preview must be inspected
247+
- **PR descriptions must include Netlify deploy preview links** for any pages being altered:
248+
- For unversioned docs (calico/, calico-enterprise/, calico-cloud/): `https://calico-docs-preview-next.netlify.app/<product>/next/<path-to-page>`
249+
- For versioned docs: `https://deploy-preview-<pr-number>--tigera-docs.netlify.app/<product>/<version>/<path-to-page>`
247250

248251
## Common Pitfalls and Solutions
249252

@@ -318,8 +321,5 @@ The build process is complex but reliable when following the documented sequence
318321
### Testing Requirements
319322
- All React components should have Jest tests in `__test__/` subdirectories
320323
- Test coverage thresholds: 85% branches/functions, 90% lines/statements
321-
- Snapshot testing used for component regression detectionctories
322-
- Test coverage thresholds: 85% branches/functions, 90% lines/statements
323-
- Snapshot testing used for component regression detectionAll React components should have Jest tests in `__test__/` subdirectories
324-
- Test coverage thresholds: 85% branches/functions, 90% lines/statements
324+
- Snapshot testing used for component regression detection
325325
- Snapshot testing used for component regression detectionesting used for component regression detection

0 commit comments

Comments
 (0)