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

Grunt i18n: unused var, ignored dirs #51

Merged
merged 2 commits into from
Sep 4, 2017
Merged

Grunt i18n: unused var, ignored dirs #51

merged 2 commits into from
Sep 4, 2017

Conversation

diggy
Copy link
Contributor

@diggy diggy commented Sep 4, 2017

  • remove unused banner variable ('banner' is assigned a value but never used.)
  • ignore .git dir
  • reference correct tests dir (php-tests > tests)

Negating globbing patterns in addtextdomain.update_all_domains work but do not seem to have any effect on the makepot task (reproducible by putting a php file with a __() call in e.g. bin and running grunt i18n --verbose). Adding an exclude param to the makepot task seems to be the only way to have the dirs and files within them effectively ignored.

@diggy
Copy link
Contributor Author

diggy commented Sep 4, 2017

Two additional notes (not in this PR):

  • yarn/npm complaining: warning some-plugin@0.1.0: No license field
  • versions for grunt and grunt-wp-i18n could be bumped

@diggy diggy changed the title Grunt i18n unused var ignored dirs Grunt i18n: unused var, ignored dirs Sep 4, 2017
@miya0001 miya0001 self-requested a review September 4, 2017 13:05
Copy link
Member

@miya0001 miya0001 left a comment

Choose a reason for hiding this comment

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

Oh, nice. Thanks!

@miya0001
Copy link
Member

miya0001 commented Sep 4, 2017

@diggy

Two additional notes (not in this PR):

yarn/npm complaining: warning some-plugin@0.1.0: No license field
versions for grunt and grunt-wp-i18n could be bumped

I think it is not important for now.
Let's open another PR, if we need. :)

Thanks!

@miya0001 miya0001 merged commit 24b32af into wp-cli:master Sep 4, 2017
danielbachhuber pushed a commit that referenced this pull request Nov 18, 2022
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.

2 participants