Skip to content
This repository has been archived by the owner on Feb 23, 2021. It is now read-only.

[+] Projects: Add gulp task which finds incorrect Smarty translations #223

Merged
merged 3 commits into from
Jun 9, 2016
Merged

Conversation

gskema
Copy link
Contributor

@gskema gskema commented May 31, 2016

No description provided.

@kpodemski
Copy link
Collaborator

aaaa 🔥 🔥 🔥 🔥 🔥

@gskema
Copy link
Contributor Author

gskema commented May 31, 2016

I'll add /modules/ path too

@gskema
Copy link
Contributor Author

gskema commented May 31, 2016

Permission to merge?

@kpodemski
Copy link
Collaborator

kpodemski commented May 31, 2016

I didn't test it but if that works this is a killer PR :)

🔥

@gskema
Copy link
Contributor Author

gskema commented May 31, 2016

Works well but sometimes there are false positive, for example if there's a } in the middle of the string then the RegEx things that's the end of the expression

@gskema gskema merged commit 234aa6e into PrestaShop:dev Jun 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants