search

formatjs / babel-plugin-react-intl

Extracts string messages from React components that use React Intl.
http://formatjs.io/react/
Other
420 stars 122 forks source link

Move 'printICUMessage(ast)' into separate project #19

Closed rightaway closed 8 years ago

rightaway commented 9 years ago

The printICUMessage(ast) function which was suggested to me in https://github.com/yahoo/intl-messageformat/issues/118 works great and is very useful as a standalone tool to convert an ICU AST into an ICU string pattern. Would you consider moving this into its own separate project so that I could depend on it directly in my package.json? Would be better so that I don't have to copy-paste the code and keep it in sync as updates are made upstream. Could be called for example something like icuAstToPattern.

rightaway commented 8 years ago

Would you accept a pull request for this? I'm happy to take it on.

ericf commented 8 years ago

Moving it to a separate repo seems like overkill to me.

What I'd like to do is move this code into the react-intl repo and setup that Git repo to support multiple npm packages. That way all the related code is in one place and CI is setup once. You're free to take this code and do what you wish in accordance to the license.

I'm going to close this issue, since I don't want to manage more Git repos.