Skip to content

Add Greptile - #2

Merged
patrickhaoy merged 1 commit into
mainfrom
pat/add-greptile
Sep 15, 2026
Merged

patrickhaoy merged 1 commit into
mainfrom
pat/add-greptile

Conversation

@patrickhaoy

Copy link
Copy Markdown

Adds .greptile/ configuration so Greptile can review this fork using its actual APIs and training code. Follows the three-file structure in UWLab's setup, with logic/syntax comments, scoped rules, and repository context files.

The rules focus on batched tensor operations, rollout and loss correctness, recurrent state, configuration/VecEnv compatibility, checkpoints and inference, distributed training, and portable dependencies. They avoid duplicating pre-commit checks or applying UWLab's simulator/task-registration/extension-version requirements to this standalone library. Ignore patterns use the newline-separated string format in Greptile's current configuration reference.

Upstream leggedrobotics/rsl_rl was checked at 857de6165c5fd479726ec8ac5c9303a497766f30. It has Copilot reviews, but no checked-in Greptile configuration or AI reviewer instructions to copy. The review context carries over its PEP 8, Google docstring, and Ruff conventions, which are also documented in this fork's README.

Validation: both JSON files parse; rule IDs are unique; all rule scopes match tracked files; all six context paths exist; git diff --cached --check passes; configured pre-commit hooks pass or appropriately skip for the three new files. No runtime code changes.

Activation: the Greptile GitHub App is installed on UW-Lab for selected repositories. This CLI session could not read the selected-repository list, so confirm UW-Lab/rsl_rl is included and enabled for reviews in Greptile. This PR supplies the review configuration; it does not change app permissions or install a workflow/API-key secret.

@patrickhaoy

Copy link
Copy Markdown
Author

@greptileai

1 similar comment
@patrickhaoy

Copy link
Copy Markdown
Author

@greptileai

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@patrickhaoy

Copy link
Copy Markdown
Author

@greptileai

@greptile-apps

greptile-apps Bot commented Sep 15, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The configuration-only PR appears safe to merge, with no concrete defects identified in the changed files.

Summary

Adds repository-specific Greptile review configuration without changing runtime code.

  • Configures logic and syntax review comments, ignore patterns, and scoped review rules.
  • Registers six repository files as review context.
  • Documents repository architecture, training contracts, validation expectations, and integration boundaries.

Reviews (1) · Last reviewed commit: "Add Greptile review configuration"

@patrickhaoy
patrickhaoy merged commit e7cd3c7 into main Sep 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant