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

Redo PR3 from phamernik/i18nchecker #1

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

bobjacobsen
Copy link
Member

This merge reproduces PR3 from phamernik/i18nchecker. That's what underlies the i18nchecker version currently used by JMRI.

Note that this does not exactly reproduce the .jar file. It's not yet clear why. Perhaps compiler version differences? They both show (via javap)

  minor version: 0
  major version: 49

due to the constraints in the build.xml file and children, but the current compile is still being done with Java 1.8.

Allow use of ResouceBundle.getBundle().getString to be counted as using
a key.
…ame line as that method (should check that getBundle is on same line as the identifier ResourceBundle).

Add check for getString() method, and treat String literals on the same line as that method as Strings needing to be translated.
Why do IDEs seems to always arbitrarily change projects?
If a logger and its methods are specified, String literals used as
parameters to the specified methods of the logger will be ignored
(eventually, this commit just specifies the properties).
@danielb987
Copy link

This PR should be closed since it's replaced by #5 but I don't have permissions to do so.

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.

3 participants