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

fix: could not configure the pattern #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sy1v4in
Copy link

@Sy1v4in Sy1v4in commented Aug 12, 2015

Transmit the configured pattern (through the options) to the Processor in order to process exactly the files defined by this supplied pattern.

…cessor in order to process exactly the files defined by this supplied pattern.
Download added a commit to Download/i18nline that referenced this pull request Nov 2, 2016
 * Applied jenseng#18
 * Upgraded dependencies
 * Use ulog for logging
 * Add some auto-config to use `src` and/or `lib` as directories if none were specified
 * Print some helpful info at startup
 * Print some more and more detailed progress info
 * Added a help screen to the CLI tool
 * Updated README.md, License info
Download added a commit to Download/i18nline that referenced this pull request Nov 2, 2016
 * Applied jenseng#18
 * Upgraded dependencies
 * Use ulog for logging
 * Add some auto-config to use `src` and/or `lib` as directories if none were specified
 * Print some helpful info at startup
 * Print some more and more detailed progress info
 * Added a help screen to the CLI tool
 * Updated README.md, License info
@Download
Copy link

@jenseng I think you should just apply this PR. It solves the issue in a simple way. Yes it's less flexible and does not support multiple processors, but it does solve a real problem with 18nliner without having to resort to writing plugins. I applied this fix in my fork and it does the job as promised.

Also, if you are working on it anyway 😄 why not add .jsx by default? It will make new user's life easier at virtually no cost.

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