Skip to content
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

MAINTAINERS file: Add new maintainer #248

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

Andersson007
Copy link
Collaborator

SUMMARY

#180 (comment)

@Andersson007
Copy link
Collaborator Author

@rsicart after accepting the invite that you should get via mail, feel free to press the "Squash and merge" green button at the bottom:) BTW we 1) always "Squash and merge" (merge commits are not allowed) 2) wait until CI pass (green, except codecov)

@rsicart rsicart merged commit d411a8e into ansible-collections:main Nov 25, 2021
Andersson007 added a commit to Andersson007/community.mysql that referenced this pull request Nov 29, 2021
Andersson007 added a commit that referenced this pull request Nov 29, 2021
* Allow the "%" character in database name (#227)

The naming rules for MySQL/MariaDB identifiers, when quoted, allow the
`%` character.

However, currently, the use of the `%` character in database names
results in mismatch or missing databases.

- Rewrite query to identify the databases in the catalog using
  `information_schema` instead of `SHOW DATABASES LIKE`
- Escape the `%` character in `CREATE DATABASE` query.

Signed-off-by: Nicolas Payart <[email protected]>
(cherry picked from commit 6b12435)

* mysql_db: Fix assert in tests suite (#239)

Signed-off-by: Nicolas Payart <[email protected]>
(cherry picked from commit 5522e45)

* mysql_db: Improve tests (#240)

- Define variables "db_names" and "db_formats" in defaults
- Use of the "vars" option in includes instead of default parameters
  that might be overridden by a previous task
- Use of the "loop" option in includes instead of duplicating include
  tasks
- Use a nested loop on db_names and db_formats in state_dump_import test

Signed-off-by: Nicolas Payart <[email protected]>
(cherry picked from commit e4de13a)

* MAINTAINERS file: Add new maintainer (#248)

(cherry picked from commit d411a8e)

Co-authored-by: Nicolas PAYART <[email protected]>
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