-
Notifications
You must be signed in to change notification settings - Fork 1
* first cut at a "dnstapir-cli edm stats" command #53
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
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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/[email protected] 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.