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.

Sign up for GitHub

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

Jump to bottom

fix mysqlimport manpage #1672

Closed
wants to merge 1 commit into from
Closed

fix mysqlimport manpage #1672

wants to merge 1 commit into from

Conversation

unki
Copy link
Contributor

@unki unki commented Sep 25, 2020

just a tiny text fix as the mysqlimport-manpage mentions mysqldump at several locations - but imho that should actually be mysqlimport.

@CLAassistant
Copy link

CLAassistant commented Sep 25, 2020 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@an3l an3l added the license-mca Contributed under the MCA label Sep 25, 2020
iangilfillan
iangilfillan reviewed Sep 25, 2020
View reviewed changes
Copy link
Contributor

@iangilfillan iangilfillan left a comment

Choose a reason for hiding this comment

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

Changes are all good, but this should probably rather be merged into 10.1.

@an3l an3l self-assigned this Oct 14, 2020
an3l pushed a commit that referenced this pull request Oct 14, 2020
@unki @an3l
mysqlimport manpage - s/mysqldump/mysqlimport/g
72cb208 
Closes PR #1672
- Note there are some legacy format not visible when invoking `man`

Reviewed by: Ian Gilfillan, Anel Husakovic
@an3l
Copy link
Contributor

an3l commented Oct 14, 2020

Thanks @unki, pushed to 10.1 with 72cb208.

@an3l an3l closed this Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iangilfillan iangilfillan iangilfillan left review comments

Assignees

@an3l an3l

Labels
license-mca Contributed under the MCA
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@unki @CLAassistant @an3l @iangilfillan