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

Added l as a valid localization function name #5

Closed
wants to merge 3 commits into from

Conversation

martingri
Copy link

It is to support the I18N plugin for Mojolicious.

@clintongormley
Copy link
Owner

Hiya Martin

Apologies for the slow response - I've been really busy. If you could add tests for the above, I'd be happy to merge it

thanks

clint

@clintongormley
Copy link
Owner

@martingri Any news on this?

@martingri
Copy link
Author

Hey Clint.

Sorry about my late reply. I'll add a test for it. I couldn't find a test for the Perl plugin module, did I miss it or should I just create a new one?

@clintongormley
Copy link
Owner

Heya Martin.

You can add them to https://github.com/clintongormley/locale-maketext-lexicon/blob/master/t/51-perlextract.t
Or even copy/paste that file to a new file and rename the _ to l

ta

clint

…to re-run the tests for multiple aliases. Added parameter $alias_terms to the run_test function to be able to specify which aliases to test for the plugin that is tested.
bk added a commit to bk/locale-maketext-lexicon that referenced this pull request Jan 7, 2015
@martingri martingri closed this Mar 9, 2023
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.

2 participants