Bug 1780020 Comment 5 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

> I think that the existing **/test*/unit*/**/ does match browser/components/about/test/unit/. My understanding is that ** will match 0 or more characters and span across directories

Hmm... when I used it for the HTTPS linter there are tests that were not excluded. 
Also in searchfox it doesn't look equivalent.
for `**/test*/unit*/**`:
https://searchfox.org/mozilla-central/search?q=&path=**%2Ftest*%2Funit*%2F**&case=false&regexp=false

for `**/test*/unit*/**/**`:
https://searchfox.org/mozilla-central/search?q=&path=**%2Ftest*%2Funit*%2F**%2F&case=false&regexp=false

But probably it is not relevant for `no-unused-vars` since its exempting the rest of xpcshell test directly and indirectly in https://searchfox.org/mozilla-central/source/.eslintrc.js#203,219-229

Thank you for adding a comment.
> I think that the existing **/test*/unit*/**/ does match browser/components/about/test/unit/. My understanding is that ** will match 0 or more characters and span across directories

Hmm... when I used it for the HTTPS linter there are tests that were not excluded. 
Also in searchfox it doesn't look equivalent.
for `**/test*/unit*/**`:
https://searchfox.org/mozilla-central/search?q=&path=**%2Ftest*%2Funit*%2F**&case=false&regexp=false

for `**/test*/unit*/**/**`:
https://searchfox.org/mozilla-central/search?q=&path=**%2Ftest*%2Funit*%2F**%2F&case=false&regexp=false

But probably it is not relevant for `no-unused-vars` since it's exempting the rest of xpcshell test directly and indirectly in https://searchfox.org/mozilla-central/source/.eslintrc.js#203,219-229

Thank you for adding a comment.
> I think that the existing **/test*/unit*/**/ does match browser/components/about/test/unit/. My understanding is that ** will match 0 or more characters and span across directories

Hmm... when I used it for the HTTPS linter there are tests that were not excluded. 
Also in searchfox it doesn't look equivalent.
for `**/test*/unit*/**`:
https://searchfox.org/mozilla-central/search?q=&path=**%2Ftest*%2Funit*%2F**&case=false&regexp=false

for `**/test*/unit*/**/**`:
https://searchfox.org/mozilla-central/search?q=&path=**%2Ftest*%2Funit*%2F**%2F&case=false&regexp=false

But probably it is not relevant for `no-unused-vars` since it's exempting the rest of xpcshell tests directly or indirectly in https://searchfox.org/mozilla-central/source/.eslintrc.js#203,219-229

Thank you for adding a comment.

Back to Bug 1780020 Comment 5