mirror of
https://github.com/FreeTubeApp/FreeTube
synced 2024-11-25 19:30:10 +01:00
3db6f437c9
* Add Stylelint plugin and rule for evaluating use of logical properties * Implement stylelint-advised use of logical properties in SCSS files * Implement stylelint-advised use of logical properties in CSS files * Implement stylelint linting before each commit to any branch of the repo * Remove other Stylelint plugins so that we can add them in later one-by-one After discussing this with the FreeTube team, it seems that we are still undecided on which rules we want to be active, including ones currently enabled. As a stopgap fix, we disabled Stylelint checking in our pre-commit Git hook and our recommended > freetube@0.19.0 lint-fix > run-p eslint-fix lint-style-fix command. With this change, we will be using Stylelint in our > freetube@0.19.0 lint-fix > run-p eslint-fix lint-style-fix command, while giving us the flexibility to add in additional desired plugins and rules as separate efforts. * Add video player to .stylelintignore * Replace non-logical property usage in templates This is not enforced by the plugin at this time. * Remove use of logical properties from ft-video-player, & allow empty input when .stylelintignore files are changed * Update ft-video-player.css to disable use of logical property linter We want to avoid introducing directionality-specific properties that could muck up the video player. * Remove redundant/unnecessary scripts in package.json
22 lines
411 B
JSON
22 lines
411 B
JSON
{
|
|
"plugins": ["stylelint-use-logical-spec"],
|
|
"overrides": [
|
|
{
|
|
"files": ["**/*.scss"],
|
|
"customSyntax": "postcss-scss",
|
|
"rules": {
|
|
"max-nesting-depth": null,
|
|
"selector-max-compound-selectors": null
|
|
}
|
|
},
|
|
{
|
|
"files": ["**/*.css"],
|
|
"rules": {
|
|
}
|
|
}
|
|
],
|
|
"rules": {
|
|
"liberty/use-logical-spec": ["always", { "except": ["float"] }]
|
|
}
|
|
}
|