docs: update dawgrun readme - #113
Conversation
WalkthroughThe Changesdawgrun documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tools/dawgrun/README.md`:
- Around line 38-44: Update the command examples around “go tool dawgrun” and
the local binary build, including the corresponding examples near the later
referenced section, from indented Markdown blocks to fenced shell code blocks.
Convert all four command examples while preserving their commands and
surrounding text so markdownlint MD046 passes.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 3ffd1eae-f0d9-4128-809a-732455254a81
📒 Files selected for processing (1)
tools/dawgrun/README.md
| From a `DAWGS` checkout, run the tool directly: | ||
|
|
||
| go tool dawgrun | ||
|
|
||
| With a customized `DAWGS` clone, for testing features, version differences, etc: | ||
| To build a local binary instead: | ||
|
|
||
| cd tools/dawgrun | ||
| just build-with-dawgs path/to/DAWGS | ||
|
|
||
| To switch the build back to mainline: | ||
|
|
||
| cd tools/dawgrun | ||
| just build-with-upstream | ||
| go build -o tools/dawgrun/dawgrun ./tools/dawgrun/cmd/dawgrun |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Use fenced code blocks for the new shell commands.
markdownlint-cli2 reports MD046 at Lines 40, 44, 297, and 301. Convert all four indented command examples to fenced code blocks so the README passes the configured Markdown lint check.
Also applies to: 294-301
🧰 Tools
🪛 markdownlint-cli2 (0.23.2)
[warning] 40-40: Code block style
Expected: fenced; Actual: indented
(MD046, code-block-style)
[warning] 44-44: Code block style
Expected: fenced; Actual: indented
(MD046, code-block-style)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@tools/dawgrun/README.md` around lines 38 - 44, Update the command examples
around “go tool dawgrun” and the local binary build, including the corresponding
examples near the later referenced section, from indented Markdown blocks to
fenced shell code blocks. Convert all four command examples while preserving
their commands and surrounding text so markdownlint MD046 passes.
Source: Linters/SAST tools
Description
Updates dawgrun documentation
Resolves: none
Type of Change
Testing
make test_allwithCONNECTION_STRINGset)Screenshots (if appropriate):
Driver Impact
drivers/pg)drivers/neo4j)Checklist
go.mod/go.sumare up to date if dependencies changedSummary by CodeRabbit