? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
26 Feb 2020

Now that we finally have scss-lint running on this repo (after several years of it being ignored) #28076 has revealed several issues that are because of an outdated linter and rules

as of scss-lint 0.59 (there is a pr for this from @ciar4n ) support has been added to test for fractional units (the fr thing used in grids)

This PR adds the test for fr units to our scss lint rules

reference https://github.com/sds/scss-lint/blob/master/config/default.yml

Testing Instructions

Code review

avatar brianteeman brianteeman - open - 26 Feb 2020
avatar brianteeman brianteeman - change - 26 Feb 2020
Status New Pending
avatar wilsonge wilsonge - change - 26 Feb 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-26 10:16:35
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 26 Feb 2020
avatar wilsonge wilsonge - merge - 26 Feb 2020
avatar wilsonge
wilsonge - comment - 26 Feb 2020

Thanks!

avatar brianteeman
brianteeman - comment - 26 Feb 2020

thanks

Add a Comment

Login with GitHub to post a comment