Closed faultymonk closed 1 year ago
If you adjust those files as well, ensure
go generate ./...
shows no changes, and sign the CLA, we can get this merged.
Thanks, updated!
BTW, I've signed the CLA, and it's not letting me re-sign. I'm unsure why the status on the check does not reflect it...
@faultymonk thanks so much, looks good to me 🚀
It looks like the CLA bot is confused because the first two commits are done with an email s[...]n@n[...]t.com (which I'm guessing may not be associated with your GitHub username?). I'll leave it up to you whether you would like these Git commits and the associated email in the history (merging this under a "trivial" update exemption to the CLA) or if you want to rewrite those commits with your GitHub associated email that the CLA bot is happy about.
Thanks again!
@bflad thanks for the catch & fixed! 🤦
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
This fixes the current import examples which uses incorrect resource names.
Community Note
Relates OR Closes #0000