Whitespace Padding ESLint Rule(s) #2574
Replies: 8 comments 1 reply
-
@pyrocat101 I think @AustinBratcher wants to take a stab at this |
Beta Was this translation helpful? Give feedback.
-
Sounds good. |
Beta Was this translation helpful? Give feedback.
-
@AustinBratcher do u still plan to tackle this? |
Beta Was this translation helpful? Give feedback.
-
@longlho I had planned on it, but I haven't been able to make the bandwidth. I know what to do, just haven't been able to make the time to do it. I'm out all next week, so I'd someone else beats me to it, that's fine by me. |
Beta Was this translation helpful? Give feedback.
-
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Beta Was this translation helpful? Give feedback.
-
I think this would be helpful still. Seems like it's just been auto-closed, rather than the need has gone away |
Beta Was this translation helpful? Give feedback.
-
@tom-nexudus it's closed bc there's not enough interest. |
Beta Was this translation helpful? Give feedback.
-
Hence my comment, indicating an interest. |
Beta Was this translation helpful? Give feedback.
-
Which package?
eslint-plugin-formatjs
Is your feature request related to a problem? Please describe.
Not related to a problem.
Describe the solution you'd like
Since the FormatJS CLI extract command removes whitespace padding, it would be helpful to add eslint rule(s) to help catch this kind of whitespace up front. Potential rules:
no-leading-whitespace
to catch whitespace at the front of the stringno-trailing-whitespace
to catch whitespace at the end of the stringno-padding-whitespace
to catch any whitespace on the ends of the stringI think the last option makes the most sense with the way the extract command currently works, but it might be helpful for some developers if the options were a bit more granular.
Describe alternatives you've considered
N/A
Additional context
N/A
Beta Was this translation helpful? Give feedback.
All reactions