eslint-plugin-i18n icon indicating copy to clipboard operation
eslint-plugin-i18n copied to clipboard

Adding two new i18n lint rules

Open ehom opened this issue 9 years ago • 5 comments

I'm interested in contributing to eslint-plugin-i18n. (I have more rules.)

Let me know if these look okay.

If so, don't merge yet since I'll have update the README.md file and the .md file for each new rule (and probably unit tests).

Thanks !!!

ehom avatar Apr 04 '16 20:04 ehom

Current coverage is 100.00%

Merging #1 into master will not affect coverage as of 1f5c792

@@            master      #1   diff @@
======================================
  Files            1       2     +1
  Stmts            6      12     +6
  Branches         1       2     +1
  Methods          0       0       
======================================
+ Hit              6      12     +6
  Partial          0       0       
  Missed           0       0       

Review entire Coverage Diff as of 1f5c792

Powered by Codecov. Updated on successful CI builds.

codecov-io avatar Apr 04 '16 20:04 codecov-io

Thanks for the PR, these look nice. I'll merge these when you complete updates.

chejen avatar Apr 05 '16 02:04 chejen

don't merge yet. one more test to submit.

ehom avatar Apr 06 '16 17:04 ehom

It would be great to see this pull request and package updated 😄

ntwb avatar Jun 28 '17 13:06 ntwb

@ehom Any chance you could update this PR please? It would be great to see these rules added

ntwb avatar Mar 13 '18 10:03 ntwb