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

Fixed whitespace issues in tslint-warnings.csv #709

Merged
merged 6 commits into from
Dec 30, 2018
Merged

Fixed whitespace issues in tslint-warnings.csv #709

merged 6 commits into from
Dec 30, 2018

Conversation

reduckted
Copy link
Contributor

PR checklist

Overview of change:

There were a couple of things contributing to the odd whitespace:

  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.
  3. Not exactly related to the odd whitespace, but the SDL report was trying to replace new lines in the metadata, but was only replacing the first one, so there were a few rules where the description spanned incorrectly multiple lines.

In addition to this, the rules directory used by the SDL report generator was wrong. It was pointing to dist/build, but the rules are in dist/src, so if you regenerated the SDL report, it deleted all of the lines for the rules in this package.

@reduckted
Copy link
Contributor Author

🤔 Hmm, I'm now wondering if the path to the rules directory was actually correct as dist/build. I was just running the generate:sdl-report script, but it seems like maybe the copy-package script is supposed to be run before it.

@reduckted reduckted changed the title 675 fix Fixed whitespace issues in tslint-warnings.csv Dec 29, 2018
@reduckted
Copy link
Contributor Author

Hmm, I'm now wondering if the path to the rules directory was actually correct as dist/build

I've reverted that change and the SDL report is still generated correctly when I run npm run test instead of just npm run generate:sdl-report. 👍

Copy link

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks @reduckted! This a great week for PRs 😄

@JoshuaKGoldberg JoshuaKGoldberg merged commit 39e3950 into microsoft:master Dec 30, 2018
@reduckted reduckted deleted the 675-fix branch December 30, 2018 10:03
@IllusionMH IllusionMH added this to the 6.1.0-beta milestone Feb 19, 2019
apawast pushed a commit to lupine86/tslint-microsoft-contrib that referenced this pull request Feb 26, 2019
* Rules directory was wrong.

* Only the first new line was being replaced with a space. Now they all are.

* Added trailing spaces after full stops in the description.

* Used string concatenation in the description to get rid of excess whitespace.

* Regenerated the SDL report.

* Reverted rules directory change.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Odd whitespace characters in tslint-warnings.csv
3 participants