-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
fix: fixed autoklose verification endpoint #3447
fix: fixed autoklose verification endpoint #3447
Conversation
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.
Minor Feedback! Rest looks ok to me.
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.
Thanks for the contribution @sahil9001 - Changes LGTM!
This counts eligible for Hacktoberfest, right? Can you add label is possible? |
@kashifkhan0771 Thanks for this, can you please review other PRs as well, and add the label, actually I don't know if I am doing this right or not. Your overview will give me an assurance if I am on the right path (eligibility) |
Nice work @sahil9001. Thanks for the review @kashifkhan0771. Merging |
Description:
Fixes #3446
Checklist:
make test-community
)?make lint
this requires golangci-lint)?