doc(profile): link to the four canonical PR Reviewer Guides#5
Open
estebanzimanyi wants to merge 1 commit into
Open
doc(profile): link to the four canonical PR Reviewer Guides#5estebanzimanyi wants to merge 1 commit into
estebanzimanyi wants to merge 1 commit into
Conversation
…MobilityPySpark Two doc-only changes folded into one commit (single-purpose: 'org-profile ecosystem-map cleanup'): 1. Adds 'Reviewing pull requests' section pointing to the canonical doc/contributing/reviewer-guide.md in each of the four core repos: MobilityDB, MobilityDuck, MobilitySpark, JMEOS. All four guides share the same structure (How to find this guide → CI legend → Dependency chains → Tier 1/2/3 → Review checklist) and are updated in the same commit as any PR queue change. 2. Removes MobilityPySpark from the active 'Application platforms' table and adds a new 'Historical / thesis codebases' section. The MobilityPySpark repo is the 2024 master's thesis codebase by L. León and is no longer part of the active binding line; the current Spark binding is the JVM-side MobilitySpark.
estebanzimanyi
force-pushed
the
doc/reviewer-guide-links
branch
from
6bc8318 to
c5570e6
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
1 participant
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.
Two scope-coherent doc-only changes to the org profile, folded into a single commit per the consolidation policy:
1. Reviewer-guide visibility wiring
Adds a "Reviewing pull requests" section pointing to the canonical
doc/contributing/reviewer-guide.mdin each of the four core repos: MobilityDB, MobilityDuck, MobilitySpark, JMEOS.All four guides share the same structure (How to find this guide → CI legend → Dependency chains → Tier 1/2/3 → Review checklist) and are updated in the same commit as any PR queue change. This section completes the visibility wiring already in place at the per-repo level (README banner +
.github/PULL_REQUEST_TEMPLATE.md).2. Retire MobilityPySpark from the active map
Removes MobilityPySpark from the "Application platforms" table and adds a new "🎓 Historical / thesis codebases" section. MobilityPySpark is the 2024 master's thesis codebase by L. León (UniPadova / ULB), built on PyMEOS, and is no longer part of the active binding line. The current Spark binding is the JVM-side MobilitySpark. The repo has been archived (read-only).
Test plan
main