Skip to content

Remove unused BassCSS responsive white space module#5585

Merged
aduth merged 1 commit intomainfrom
aduth-rm-basscss-responsive-white-space
Nov 5, 2021
Merged

Remove unused BassCSS responsive white space module#5585
aduth merged 1 commit intomainfrom
aduth-rm-basscss-responsive-white-space

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Nov 5, 2021

Why: Because none of these classes are currently in use.

I noticed that all classes in this file are already covered by banned ERBLint deprecated class pattern.

- '((sm|md|lg)-)?[mp][trblxy]?(n?[0-6]|-auto)'

https://regex101.com/r/nl5J6U/1

Source: https://github.com/basscss/basscss-sass/blob/v3.0.0/_responsive-white-space.scss

**Why**: Because none of these classes are currently in use.
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@aduth aduth merged commit c5b18d8 into main Nov 5, 2021
@aduth aduth deleted the aduth-rm-basscss-responsive-white-space branch November 5, 2021 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants