Conversation
… Unitree Go2 Navigation & Exploration Beta Pre-Release v0.0.8: Unitree Go2 Navigation & Exploration Beta, Transport Updates, Documentation updates, Rerun fixes, Person follow, Readme updates ## What's Changed * Small docs clarification about stream getters by @leshy in #1043 * Fix split view on wide monitors by @jeff-hykin in #1048 * Docs: Install & Develop by @jeff-hykin in #1022 * Add uv to nix and fix resulting problems by @jeff-hykin in #1021 * v0.0.8 by @paul-nechifor in #1050 * Style changes in docs by @paul-nechifor in #1051 * Revert "Add uv to nix and fix resulting problems" by @leshy in #1053 * Transport benchmarks + Raw ros transport by @leshy in #1038 * feat: default to rerun-web and auto-open browser on startup (browser … by @Nabla7 in #1019 * bbox detections visual check by @leshy in #1017 * fix: only auto-open browser for rerun-web viewer backend by @Nabla7 in #1066 * move slow tests to integration by @paul-nechifor in #1063 * Streamline transport start/stop methods by @Kaweees in #1062 * Person follow skill with EdgeTAM by @paul-nechifor in #1042 * fix: increase costmap floor z_offset to avoid z-fighting by @Nabla7 in #1073 * Fixed issue #1074 by @alexlin2 in #1075 * ROS transports initial by @leshy in #1057 * Fix System Config Values for LCM on MacOS and Refactor by @jeff-hykin in #1065 * SHM Transport basic fixes by @leshy in #1041 * commented out Mem Transport test case by @leshy in #1077 * Docs/advanced streams update 2 by @leshy in #1078 * Fix more tests by @paul-nechifor in #1071 * feat: navigation docker updates from bona_local_dev by @baishibona in #1081 * Fix missing dependencies by @Kaweees in #1085 * Release readme fixes by @spomichter in #1076 ## New Contributors * @baishibona made their first contribution in #1081 **Full Changelog**: v0.0.7...v0.0.8
…HTTPS from SSH, get_data change, LFS changes v0.0.9 Release Patch: Git clone change to HTTPS from SSH, get_data change, LFS changes
Added a draft pull request template with sections for summary, features, scope, and effort estimates.
Removed draft notice from pull request template
Greptile OverviewGreptile SummaryThis PR adds a comprehensive pull request template to standardize PR submissions across the repository. The template includes well-structured sections for description, key features, scope definition, stack categorization (Navigation/Perception/Manipulation), breaking changes documentation, and effort estimation. Key additions:
The template follows GitHub best practices and will improve PR quality and review efficiency by ensuring consistent information is provided upfront. Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant GH as GitHub
participant Template as PR Template
participant Reviewer as Reviewer
Dev->>GH: Create new pull request
GH->>Template: Load pull_request_template.md
Template->>GH: Populate PR description
Dev->>GH: Fill in template sections
Note over Dev,GH: - Short Description<br/>- Key Feature<br/>- Feature Scope<br/>- Category (Nav/Perception/Manip)<br/>- Breaking Changes<br/>- Effort Estimate
Dev->>GH: Submit PR
GH->>Reviewer: Notify with structured info
Reviewer->>GH: Review with context
Reviewer->>GH: Approve/Request changes
|
Updated the pull request template to include sections for problem, solution, breaking changes, and testing instructions.
This script runs various checks for Python, Node, and CMake projects, including linting, testing, and building.
Updated the Contributor License Agreement to clarify project ownership and licensing terms.
There was a problem hiding this comment.
Summary of changes in this PR
-
Added a CLA (
CLA.md) to define contribution terms and ensure proper licensing of external contributions. -
Added a PR test script (
scripts/test_pr.sh) that runs basic checks and tests to verify changes locally and in CI. -
Added a GitHub Actions workflow (
.github/workflows/pr.yml) to automatically run the PR test script on every pull request. -
Updated the pull request template (
.github/pull_request_template.md) to:- Focus on Problem / Solution / Breaking Changes / How to Test
- Require reproducible testing steps
- Include CLA acknowledgment
Goal:
These changes establish a standard open-source workflow by adding contributor license, reproducible testing, and automated CI checks for all pull requests.
christiefhyang
left a comment
There was a problem hiding this comment.
Updated workflow permissions
Pull Request Template Summary