* first cut at a "dnstapir-cli edm stats" command#53
Conversation
📝 WalkthroughWalkthroughRegisters a new Changes
Sequence Diagram(s)(omitted — changes are small and don't introduce multi-component control flow) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@go.mod`:
- Around line 5-6: The go.mod currently has a local replace directive "replace
github.com/dnstapir/tapir => ../tapir" which breaks CI; remove that replace
directive from go.mod before merging and either update the require for
github.com/dnstapir/tapir to the published version you need or keep the PR as a
draft until the tapir changes are published so CI can resolve the dependency.
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (2)
cmd/root.gogo.mod
🧰 Additional context used
🪛 GitHub Actions: Build
cmd/root.go
[error] 16-16: Build failed: replacement directory '../tapir' does not exist for github.com/dnstapir/tapir@v0.0.0-20251029102003-58142cabe113 during go build.
🔇 Additional comments (1)
cmd/root.go (1)
57-59: EdmCmd registration follows the established pattern.The command registration at line 58 is consistent with how
PopCmd,DawgCmd,ApiCmd,FilterlistsCmd,EnrollCmd, andRenewCmdare added. The syntax is identical and the import source matches other commands. The go.mod replacement forgithub.com/dnstapir/tapiris properly configured pointing to the local../tapirdirectory.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
|
Don't forget to update techdocs |
Summary by CodeRabbit
New Features
Chores
✏️ Tip: You can customize this high-level summary in your review settings.