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

Add --location flag to omit source code references #264

Merged
merged 1 commit into from
Jul 19, 2021

Conversation

swissspidy
Copy link
Member

Fixes #240

@swissspidy swissspidy requested a review from a team as a code owner July 5, 2021 14:56
@swissspidy swissspidy enabled auto-merge July 7, 2021 13:52
@swissspidy swissspidy merged commit 1095a00 into master Jul 19, 2021
@delete-merged-branch delete-merged-branch bot deleted the add/no-location branch July 19, 2021 16:33
@schlessera schlessera added this to the 2.2.9 milestone Jul 19, 2021
schlessera pushed a commit that referenced this pull request Jan 6, 2022
Add new flag to omit source code references
@schlessera schlessera changed the title Add new flag to omit source code references Add --location flag to omit source code references Jan 25, 2022
@luiseduardobraschi
Copy link

I'm getting unknown --location parameter in version 2.6.0. When I use wp i18n make-pot it doesn't add the #file:line references. Should I open an issue?

@swissspidy
Copy link
Member Author

Works fine for me:

$ wp i18n make-pot --location
usage: wp i18n make-pot <source> [<destination>] [--slug=<slug>] [--domain=<domain>] [--ignore-domain] [--merge[=<paths>]] [--subtract=<paths>] [--subtract-and-merge] [--include=<paths>] [--exclude=<paths>] [--headers=<headers>] [--location] [--skip-js] [--skip-php] [--skip-block-json] [--skip-theme-json] [--skip-audit] [--file-comment=<file-comment>] [--package-name=<name>]

Sounds like you might have an old version of the command installed. Check with wp package list.

If that doesn't help, please reach out via Slack instead, as this is most likely an issue with your setup and not a bug.

@luiseduardobraschi
Copy link

Oh, boy. That was it. I totally forgot it was a whole different package. Thank you very much.

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

Successfully merging this pull request may close these issues.

Can I disable/remove paths comments?
3 participants