I tried to use write_posts_to_csv recently and got an error message because I was missing the "matching" argument. I wanted to keep all the posts so I ended up using a regex code to match anything.
But it seemed a bit unnecessary so I looked at the code and it seems the CLI already has a default.
So I added the default in the function itself.
Some follow-up questions to improve on the PR:
Should we remove the default value in the CLI parser? It's now redundant.
Is .+ the best regex to match absolutely anything? It seems like it could ignore newlines, which might not be a possible case for Facebook post's text?
Hi,
I tried to use
write_posts_to_csvrecently and got an error message because I was missing the "matching" argument. I wanted to keep all the posts so I ended up using a regex code to match anything.But it seemed a bit unnecessary so I looked at the code and it seems the CLI already has a default.
So I added the default in the function itself.
Some follow-up questions to improve on the PR:
.+the best regex to match absolutely anything? It seems like it could ignore newlines, which might not be a possible case for Facebook post's text?