Skip to content

Commit

Permalink
Fix dot folders and move stylelint fix
Browse files Browse the repository at this point in the history
  • Loading branch information
myovchev committed Nov 8, 2024
1 parent cb8a7f3 commit 2bd2aa3
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
6 changes: 1 addition & 5 deletions .stylelintrc
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
{
"extends": "stylelint-config-apostrophe",
"//": "declaration-strict-value issue: https://github.com/AndyOGo/stylelint-declaration-strict-value/issues/379",
"rules": {
"scale-unlimited/declaration-strict-value": null
}
"extends": "stylelint-config-apostrophe"
}
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
* Widget players are now properly invoked when they arrive later in the page load process.
* Fix permission grid tooltip display.
* Fixes a bug that crashes external frontend applications.
* Fixes a false positive warning for module not in use for project level submodules (e.g. `widges/module.js`).
* Fixes a false positive warning for module not in use for project level submodules (e.g. `widges/module.js`) and dot-folders (e.g. `.DS_Store`).

### Adds

Expand Down
3 changes: 3 additions & 0 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -699,6 +699,9 @@ async function apostrophe(options, telemetry, rootSpan) {
}
}
async function testDir(name) {
if (name.startsWith('.')) {
return;
}
// Projects that have different theme modules activated at different times
// are a frequent source of false positives for this warning, so ignore
// seemingly unused modules with "theme" in the name
Expand Down

0 comments on commit 2bd2aa3

Please sign in to comment.