Skip to content

docs: update dawgrun readme - #113

Merged
seanjSO merged 1 commit into
mainfrom
seanj/dawgrun-readme-cleanup
Aug 7, 2026
Merged

docs: update dawgrun readme#113
seanjSO merged 1 commit into
mainfrom
seanj/dawgrun-readme-cleanup

Conversation

@seanjSO

@seanjSO seanjSO commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Description

Updates dawgrun documentation

Resolves: none

Type of Change

  • Chore (a change that does not modify the application functionality)
  • Bug fix (a change that fixes an issue)
  • New feature / enhancement (a change that adds new functionality)
  • Refactor (no behaviour change)
  • Test coverage
  • Build / CI / tooling
  • Documentation

Testing

  • Unit tests added / updated
  • Integration tests added / updated
  • Full test suite run (make test_all with CONNECTION_STRING set)

Screenshots (if appropriate):

Driver Impact

  • PostgreSQL driver (drivers/pg)
  • Neo4j driver (drivers/neo4j)

Checklist

  • Code is formatted
  • All existing tests pass
  • go.mod / go.sum are up to date if dependencies changed

Summary by CodeRabbit

  • Documentation
    • Added instructions for running the tool directly and compiling a local binary.
    • Simplified build guidance by removing specialized checkout instructions.
    • Added troubleshooting guidance for default graph initialization failures, including available initialization options.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The dawgrun README now documents direct tool execution, local binary compilation, and troubleshooting steps for PostgreSQL databases without a default graph.

Changes

dawgrun documentation

Layer / File(s) Summary
Build and troubleshooting guidance
tools/dawgrun/README.md
The README documents go tool dawgrun and local go build usage. It also explains -init-graph and -default-graph options for PostgreSQL databases without a default graph.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

A rabbit found the build path clear,
go tool dawgrun hops near.
If graphs are missing in the den,
-init-graph starts them again.
-default-graph picks the right one.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies the documentation update to the dawgrun README.
Description check ✅ Passed The description includes the required sections and identifies this as a documentation-only change with no testing or driver impact.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch seanj/dawgrun-readme-cleanup

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai 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.

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

📥 Commits

Reviewing files that changed from the base of the PR and between bd64d91 and 088fafc.

📒 Files selected for processing (1)
  • tools/dawgrun/README.md

Comment thread tools/dawgrun/README.md
Comment on lines +38 to +44
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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 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

@computator computator left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for updating this!

@seanjSO
seanjSO merged commit 119ca13 into main Aug 7, 2026
9 checks passed
@seanjSO
seanjSO deleted the seanj/dawgrun-readme-cleanup branch August 7, 2026 15:25
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.

2 participants