Merged
Conversation
- Added a `help` target that lists available Makefile targets with descriptions. - Added and standardized `##` comments for all targets to provide concise help messages. - Set `.DEFAULT_GOAL` to `help` for improved usability. - Removed redundant comments now covered by help descriptions. This enhances developer experience and discoverability of Makefile functionality.
…riant handling - Introduce TESTNET_VARIANT variable (default: conway_fast) for flexible testnet setup. - Add make targets: prepare-cluster-scripts, start-cluster, stop-cluster for local testnet cluster management. - Rename update-lockfile to update-uv-lock for clarity. - Add update-flake-lock target for Nix flake updates. - Update README to document new make targets and usage, replacing manual cluster script calls with make commands. - Update ai_run.sh to use new update-uv-lock target.
Contributor
There was a problem hiding this comment.
Pull request overview
Updates local developer tooling and documentation for running tests and managing common tasks (linting, lockfiles, and local testnet cluster lifecycle).
Changes:
- Refresh README instructions to emphasize containerized local runs via
runner/runc.sh, and to usemake start-cluster/stop-cluster. - Enhance
Makefilewith a help target, target descriptions, cluster lifecycle targets, and renameupdate-lockfile→update-uv-lock. - Update
ai_run.shallowlist to match the renamed make target.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| README.md | Updates local run and cluster workflow documentation; adds container prefix tip. |
| Makefile | Adds self-documenting targets, cluster start/stop helpers, lockfile and flake update targets. |
| ai_run.sh | Updates allowed make subcommands to reflect renamed target(s). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Streamline instructions for running tests locally and in containers - Emphasize preferred use of `runner/runc.sh` for containerized runs - Simplify Nix usage section and clarify alternatives - Update example commands for consistency and clarity - Add tip about prefixing commands with `./runner/runc.sh --` for containerization
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
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.
No description provided.