Skip to content

Conversation

matt-horwood-mayden
Copy link

SUMMARY

This might add a use comment, I dont know what Im doing and have no context of how this works

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

proxysql_mysql_users

ADDITIONAL INFORMATION

add comment to user
Copy link
Contributor

@Andersson007 Andersson007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matt-horwood-mayden hello, thanks for the PR!
Could you please:

UPDATE: if you have any questions, please ask

Thanks

Co-authored-by: Andrew Klychkov <[email protected]>
@Andersson007
Copy link
Contributor

Andersson007 commented Jul 9, 2025

@matt-horwood-mayden also a couple of bits:

  • You can run the tests locally easily before pushing, see https://docs.ansible.com/ansible/devel/community/create_pr_quick_start.html
  • You could mark the PR as draft as long as the CI is red, so that people subscribed to the repo don't get a notification about every push. You can click "ready for review" button once it's ready. We'll get notified about comments here anyway, if you have any questions.

@matt-horwood-mayden matt-horwood-mayden marked this pull request as draft July 9, 2025 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants