Skip to content

Commit 29160dc

Browse files
alishaz-polymathalishaz-polymath
authored
Merge branch 'main' into feat/private-links-2
2 parents e29dc1b + 3ab6767 commit 29160dc

53 files changed

Lines changed: 2425 additions & 704 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.cursor/rules/review.mdc‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,3 +19,10 @@
1919
- "summary" fields do not end with a period
2020
- "summary" fields are written in proper American english
2121
- When changes to API v2 or v1 are made, ensure there are no breaking changes on existing endpoints. Instead, there needs to be a newly versioned endpoint with the updated functionality but the old one must remain functional
22+
- For large pull requests (>500 lines changed or >10 files touched), advise splitting into smaller, focused PRs:
23+
- Split by feature boundaries: separate different features or user stories
24+
- Split by layer/component: frontend changes, backend changes, database migrations, and tests in separate PRs
25+
- Split by dependency chain: create PRs that can be merged sequentially
26+
- Split by file/module: group related file changes together
27+
- Suggested pattern: Database migrations → Backend logic → Frontend components → Integration tests
28+
- Benefits: easier review, faster feedback, reduced conflicts, easier rollback, better history

‎.yarn/versions/a6aae389.yml‎

Whitespace-only changes.

0 commit comments

Comments
 (0)