Add Greptile review config with UW Lab coding conventions - #34
Merged
Conversation
Add a .greptile/ directory so Greptile reviews PRs against the project's conventions instead of only generic checks: - config.json: restrict comments to logic/syntax (formatting is handled by pre-commit), set review priorities, and add scoped rules for extension version + CHANGELOG bumps, cross-script CLI flag consistency, vectorized mdp terms, gym registration entry points, self-contained task packages, new-task docs, optional research dependencies, submodule policy, machine-specific paths, committed artifacts, type hints, TYPE_CHECKING imports, bash shebangs, and new dependencies. - rules.md: coding conventions adapted from Isaac Lab's contributing guide (file/class structure, type hints and docstrings, manager-based env conventions, task registration, adding a new task or publication, changelog format, tests, portability). - files.json: point the reviewer at the PR template, pre-commit config, and .gitattributes for context.
Run ./uwlab.sh --format over the repository and commit the result so the pre-commit CI check passes. Whitespace, import order, and end-of-file fixes only; no functional changes.
github-actions
Bot
added
documentation
Greptile SummaryAdds repository-level Greptile configuration and UW Lab review guidance.
Confidence Score: 5/5The PR appears safe to merge with no concrete actionable issues identified. The changes add review-tool configuration and documentation, and the investigated claims about repository conventions did not establish a reachable incorrect behavior.
|
| Filename | Overview |
|---|---|
| .greptile/config.json | Defines review behavior, priorities, ignore patterns, and repository-specific rules without an established actionable defect. |
| .greptile/files.json | Registers three relevant repository files as additional review context. |
| .greptile/rules.md | Documents UW Lab’s coding, task, dependency, testing, and release conventions consistently with the checked repository context. |
Reviews (1): Last reviewed commit: "Add Greptile review config with UW Lab c..." | Re-trigger Greptile
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
2 participants
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.
Description
Add a .greptile/ directory for Greptile to review PRs.
Type of change
Checklist
pre-commitchecks with./uwlab.sh --formatconfig/extension.tomlfileCONTRIBUTORS.mdor my name already exists there