doc: add PR Reviewer Guide + visibility wiring (uniform with MobilityDB/MobilitySpark)#117
Open
estebanzimanyi wants to merge 1 commit intomainfrom
Open
doc: add PR Reviewer Guide + visibility wiring (uniform with MobilityDB/MobilitySpark)#117estebanzimanyi wants to merge 1 commit intomainfrom
estebanzimanyi wants to merge 1 commit intomainfrom
Conversation
3cc7344 to
4256154
Compare
5 tasks
…policy - Adds `doc/contributing/reviewer-guide.md` — single source of truth for reviewing open PRs. Canonical structure shared with MobilityDB (PR #931) and MobilitySpark (PR #8): How to find this guide → CI legend (5 symbols incl.⚠️ ) → Dependency chains → Tier 1/2/3 → Review checklist. - Wires visibility for reviewers landing on the repo: - `.github/PULL_REQUEST_TEMPLATE.md` — banner appears in every new PR. - `README.md` — "For contributors and reviewers" section, links to the reviewer guide; mentions canonical cross-repo structure. - Establishes the rule: every PR commit that changes queue state (opens/closes PR, discovers a dependency) updates the guide in the same commit. Once this lands and companion PRs land on MobilityDB/MobilitySpark, anyone landing in any of the three platform repos finds the same canonical guide at the same path with consistent visibility through the README and the PR template.
4256154 to
770bfef
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds
doc/contributing/reviewer-guide.mdfor MobilityDuck, structurally identical to the MobilityDB and MobilitySpark guides at the same path:Wires visibility so reviewers landing on the repo find the guide quickly:
.github/PULL_REQUEST_TEMPLATE.md— every new PR description starts with a banner pointing to the guideREADME.md— new "For contributors and reviewers" section right before the Requirements sectionThe guide is updated in the same commit as any PR queue change (one-liner status change is enough; a fuller rewrite is needed when the dependency graph changes).
Cross-repo coordination
Companion PRs landing the same uniformization + visibility wiring:
Once all three merge, anyone landing in any of the three platform repos finds the same canonical guide at the same path.
Test plan