rootstrap / i18n_linter

Rails i18n Linter Gem
https://rootstrap.com
MIT License
44 stars 2 forks source link

Exclude detected warnings #7

Closed f19ps closed 5 years ago

f19ps commented 5 years ago

Make it possible to exclude detected warnings from configuration file

The way to indicate the exclusion of warnings is as follows:

For example, if the linter detects that the file app/models/user.rb has a string to localize in line 10 and column 12, it should be done by adding app/models/user.rb:10:12 to Warnings: Exclude: [] in the configuration file:

Warnings: Exclude: [ 'app/models/user.rb:10:12' ]

santib commented 5 years ago

@fpariani Apart from that, I just noticed you are merging to a develop branch. It doesn't work like that for gems. You should be merging to master because gems work with releases and everyone knows the master branch is unstable, that's why you don't need the develop branch.