Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Odd whitespace characters in tslint-warnings.csv #675

Closed
JoshuaKGoldberg opened this issue Dec 14, 2018 · 1 comment · Fixed by #709
Closed

Odd whitespace characters in tslint-warnings.csv #675

JoshuaKGoldberg opened this issue Dec 14, 2018 · 1 comment · Fixed by #709
Labels
Domain: Tooling Repository tasks around improving source tooling. Status: Needs Investigation We're not sure whether this a good thing to do.
Milestone
6.1.0-beta

Comments

@JoshuaKGoldberg
Copy link

See conversation in #674.

Why there are no spaces after .? Like that `role`.Many, particular roles.Some , to `role='link'`.A reference.

Change above contain spaces (to many?). Probably requires build script update.
@JoshuaKGoldberg JoshuaKGoldberg added Status: Needs Investigation We're not sure whether this a good thing to do. Domain: Tooling Repository tasks around improving source tooling. labels Dec 14, 2018
@JoshuaKGoldberg JoshuaKGoldberg mentioned this issue Dec 14, 2018
Merged
4 tasks
@reduckted reduckted mentioned this issue Dec 29, 2018
Merged
1 task
@reduckted
Copy link
Contributor

From the PR (#709):

  1. react-a11y-role-supports-aria-props was using string concatenation in the description, but didn't have trailing spaces at the end of each string.
  2. react-a11y-role was using a template literal with a new line and indentation in the middle.

@IllusionMH IllusionMH added this to the 6.1.0-beta milestone Feb 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Domain: Tooling Repository tasks around improving source tooling. Status: Needs Investigation We're not sure whether this a good thing to do.
Projects
None yet
Milestone
6.1.0-beta
Development

Successfully merging a pull request may close this issue.

Fixed whitespace issues in tslint-warnings.csv
3 participants
@IllusionMH @JoshuaKGoldberg @reduckted