joker: Modernize record conversion - #4559
Conversation
|
Dear @atrull, GitHub could not add you as a reviewer, so I sent you an invitation for triage access to this repository. Please accept the invitation when convenient, then take a look at this PR and run the provider integration test. Thank you! |
|
Hi @TomOnTime I'll try to get to it this week. Thanks for reaching out. |
Hi @TomOnTime I ran: integrationTest % go test -v -args -verbose -profile JOKER everything passed/skipped bar: Here's the full log of that: --- FAIL: TestDNSProviders (182.26s) |
|
Thanks for the fast reply! We can ignore that last test. It's basically a no-op. Best, |
|
I approve, I'm just unable to click approve or find a way via the web interface (Github UX en**itification?) |
|
Thanks! (yeah, it may be github, it may be that the "triage" role can't approve PRs) |
Dear @atrull. I could use your help. I'm doing upgrades to the code base and I have no way to test your provider. Please run the integration test and let me know the result. How to run integration tests is here: https://docs.dnscontrol.org/developer-info/integration-tests
Summary
Validation
../../bin/is_modern.sh(Steps 1-7 clean)go test ./providers/jokerbin/generate-all.shgo test ./...