Skip to content

Replace _s by theme-slug for text_domain value in phpcs ruleset file#56

Open
truongwp wants to merge 1 commit into
Automattic:masterfrom
truongwp:patch-2
Open

Replace _s by theme-slug for text_domain value in phpcs ruleset file#56
truongwp wants to merge 1 commit into
Automattic:masterfrom
truongwp:patch-2

Conversation

@truongwp

Copy link
Copy Markdown
Contributor

No description provided.

@truongwp

truongwp commented Jul 7, 2017

Copy link
Copy Markdown
Contributor Author

Related #54

@jrfnl

jrfnl commented Jul 7, 2017

Copy link
Copy Markdown

Related ? You mean duplicate, I presume ?

@grappler

grappler commented Jul 8, 2017

Copy link
Copy Markdown

@jrfnl #56 is a PR and #54 is an issue.

@jrfnl

jrfnl commented Jul 8, 2017

Copy link
Copy Markdown

@grappler Thanks for putting me right.

In that case, shouldn't it be: Fixes #54 ? 👍

@truongwp

truongwp commented Jul 8, 2017

Copy link
Copy Markdown
Contributor Author

@jrfnl Like you said in the issue:

The _s in the below snippet is not being replaced by the generator at this time.
Once WPCS 0.12.0 comes out, a second customization will be added, which will also take _s as an property value, so let's keep this issue open until this is handled correctly.

So I'm not sure this will be merged or wait for WPCS 0.12.0, so I am not sure it should be "Fixes"

@jrfnl

jrfnl commented Jul 8, 2017

Copy link
Copy Markdown

@truongwp Partially fixes maybe ?
Thanks for creating the PR!

@davidakennedy

Copy link
Copy Markdown
Contributor

Thanks for the pull request @truongwp! It may be better to go ahead and wait for WPCS 0.12.0 so everything can be fixed at once.

@MehbubRashid

Copy link
Copy Markdown

Hi @davidakennedy this problem still existing in 2021

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants