Fix/3717 filter table on diesel migration --diff-schema

[gmanninglive]

Fixes: #3717

I have made changes to allow diesel migration --diff-schema to accept filter args matching diesel print-schema

I came across a few issues while working on this:

1. I can't think of how to move the filter attribute from PrintSchema to Config without a breaking change
2. The documentation for table-names arg seems incorrect to me, It currently states (default only-tables if not empty). But I'm pretty sure it defaults to None currently

TODO:

1. Update documentation where necessary

I'll add some more info to this PR description before requesting review

[weiznich]

Thanks for working on this. The implementation looks really good 👍 I have one minor question otherwise this is fine from my side.

[weiznich]

I've rebased this branch to pull in the changes from #3729 to fix the broken tests.

[gmanninglive]

@weiznich would you like me to change this to:

Suggested change

	.help("Table names to filter (default only-tables if not empty)."),
	.help("Table names to filter"),

Because as far as I can see, filtering so far has only worked by specifying -o or -e
the change would also be made for print-schema below

[weiznich]

I'm in favor of this change (also for the print-schema) part.

[gmanninglive]

Sorry for the delay I've just made this change. I think this should be good to go now, as long as you're happy with it.

[weiznich]

No worries from taking some time. We all work on this in our free time, so it's fine to take as much time as needed. There is always real life, which is more important than this.

[gmanninglive]

I've rebased this branch to pull in the changes from #3729 to fix the broken tests.

Thanks for rebasing this!, I've left a couple comments and will set ready for review. Thanks for approving already too 😄

[weiznich]

Thanks 👍