-
Notifications
You must be signed in to change notification settings - Fork 1
Implement working uv/pytest environment in copilot-setup-steps workflow and add documentation #126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
b958c46
52756d0
f17e693
97ecfcb
910c40e
ec99005
fb2f320
8248047
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,35 @@ | ||
| # GitHub Copilot Development Environment | ||
|
|
||
| **Primary agent instructions**: See [`AGENTS.md`](../AGENTS.md) for complete repository context. | ||
|
|
||
| ## Environment Setup | ||
|
|
||
| GitHub Copilot agents use uv/Python (not Nix) because Nix causes Bash tool calls to hang. Setup is handled by `.github/workflows/copilot-setup-steps.yml`: | ||
|
|
||
| ```bash | ||
| uv sync --extra dev | ||
| ``` | ||
|
|
||
| ## Available Tools | ||
|
|
||
| All tools must be run via `uv run`: | ||
|
|
||
| | Tool | Command | | ||
| |------|---------| | ||
| | **pytest** | `uv run pytest tests/ -v` | | ||
| | **ruff** | `uv run ruff check src/` | | ||
| | **mypy** | `uv run mypy src/` | | ||
| | **deepwork** | `uv run deepwork` | | ||
|
|
||
| ## Running Tests | ||
|
|
||
| ```bash | ||
| # All tests | ||
| uv run pytest tests/ -v | ||
|
|
||
| # Specific test file | ||
| uv run pytest tests/test_example.py -v | ||
|
|
||
| # With coverage | ||
| uv run pytest tests/ --cov=deepwork --cov-report=html | ||
| ``` |
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @copilot create .github/copilot-instructions.md that indicates how development can be done given these setup steps. It should only really document this leaving the primary context defined in AGENTS.md
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Created in ec99005. The
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot make this terse, it should only document the AGENTS.md and the tools available as well as instructions for running tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made terse in fb2f320. The file now only includes:
Reduced from 128 lines to 35 lines.