Skip to content

Commit a20aa7e

Browse files
clarkbwjasonLaster
authored andcommitted
update lint script to better capture directory layout (firefox-devtools#2086)
* update style lint locations scan src/components/ and 2 levels deeper than that for CSS files. * update the ignore file * fix styles to pass linting * quote the param to make a glob work * quote the lint-js glob param to make it work * update the lint-md paths
1 parent e6f7702 commit a20aa7e

File tree

5 files changed

+9
-11
lines changed

5 files changed

+9
-11
lines changed

.stylelintignore

+3-2
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,3 @@
1-
src/components/SplitBox.css
2-
src/components/reps.css
1+
src/components/Editor/codemirror-mozilla.css
2+
src/components/shared/SplitBox.css
3+
src/components/shared/reps.css

package.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,9 @@
2020
"flow": "flow",
2121
"links": "ls -l node_modules/ | grep ^l || echo 'no linked packages'",
2222
"lint": "yarn run lint-css -s && yarn run lint-js -s && yarn run lint-md",
23-
"lint-css": "stylelint src/components/*.css",
24-
"lint-js": "eslint src",
25-
"lint-md": "remark -q src docs README.md CONTRIBUTING.md",
23+
"lint-css": "stylelint \"src/components/**/*.css\"",
24+
"lint-js": "eslint \"src/**/*.js\"",
25+
"lint-md": "remark -q *.md src/**/*.md configs/*.md docs/**/*.md",
2626
"lint-fix": "yarn run lint-js -- --fix",
2727
"test": "node src/test/node-unit-tests.js",
2828
"test-all": "yarn run test; yarn run lint; yarn run flow",

src/components/SecondaryPanes/CommandBar.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ html .command-bar > button:disabled {
4646
}
4747

4848
.command-bar button.pause-exceptions {
49-
margin-inline-start: .5em;
49+
margin-inline-start: 0.5em;
5050
}
5151

5252
.command-bar .subSettings {

src/components/SecondaryPanes/Frames.css

+2-2
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,8 @@
4949

5050
.frames ul li:hover,
5151
.frames ul li:focus {
52-
background-color: var(--theme-toolbar-background-alt);
53-
outline: none;
52+
background-color: var(--theme-toolbar-background-alt);
53+
outline: none;
5454
}
5555

5656
.frames ul li.selected {

src/components/shared/Button/Close.css

-3
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,6 @@
3636
height: 40px;
3737
}
3838

39-
.close-btn-big path {
40-
}
41-
4239
.close-btn-big .close {
4340
cursor: pointer;
4441
display: inline-block;

0 commit comments

Comments
 (0)