lsegal/yard — pull request #527 by kou (2012-04-30 14:37:12+00:00)
Title: i18n: move YARD::CLI::I18n::* to YARD::I18n::*

Those changes are corresponding to 1. and 3. in #395:

https://github.com/lsegal/yard/pull/395#issuecomment-5348096

> 1. I'd like it if you could move your CLI::I18n subclasses into a
> YARD::I18n namespace. I'm not a fan of classes inside of classes,
> organizationally speaking, and it's hard to promote component reuse
> for these classes if they're inside of a CLI namespace. Presumably
> someone might want to use the PotGenerator class directly, for
> instance, and they shouldn't have to go through the CLI. So, I've
> marked those classes as `@private` for now so that nobody uses them
> from that namespace. Once you move them into a more appropriate
> location, you can remove the `@private` declaration and we can support
> them as a public API.
>
> 2. ...
>
> 3. Ditto for moving the tests for the inner classes like
> PotGenerator and Test into their own files. I'm also not a fan of
> multiple classes in a single _spec.rb file.