Skip to content

Introduce wp db ack #7

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
danielbachhuber opened this issue Apr 4, 2017 · 6 comments · Fixed by #29
Closed

Introduce wp db ack #7

danielbachhuber opened this issue Apr 4, 2017 · 6 comments · Fixed by #29

Comments

@danielbachhuber
Copy link
Member

Find a specific string in the database, a la https://github.com/runcommand/db-ack

@gitlost
Copy link
Contributor

gitlost commented Jul 28, 2017

Is there anything other than basically a straight merge required here? As mentioned in wp-cli/search-replace-command#1 (comment) I'd like to do this.

@danielbachhuber
Copy link
Member Author

Is there anything other than basically a straight merge required here?

Probably add a few more tests and clean the code up a bit. I had also wanted to add runcommand/db-ack#5

@gitlost
Copy link
Contributor

gitlost commented Jul 28, 2017

Do you want to do this instead?

Re runcommand/db-ack#5 I think just merging anyway and iterating on that would be good.

@danielbachhuber
Copy link
Member Author

Do you want to do this instead?

It's not high on my list of things to do. If you want to pick it up, feel free to go for it.

@miya0001
Copy link
Member

It is a personal question, what does ack mean?

@miya0001
Copy link
Member

Ah, I see http://beyondgrep.com/.
I saw dictionary... 😓

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants