-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
core(stacks): actually support joomla and october cms #11788
Conversation
@@ -22,11 +22,7 @@ describe('stack-packs lib', () => { | |||
const result = lighthouseStackPacksDep | |||
.filter(p => !stackPacksLib.stackPacksToInclude.find(p2 => p2.packId === p.id)) | |||
.map(p => p.id); | |||
expect(result).toMatchInlineSnapshot(` | |||
Array [ | |||
"joomla", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lolll
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ya it didnt work for its intended purpose lol
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well at least we know it works :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we convert this to toEqual([]);
with a comment that we shouldn't bother updating a stack pack until detectors are in place? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change itself is fine, test improvements though :)
@@ -22,11 +22,7 @@ describe('stack-packs lib', () => { | |||
const result = lighthouseStackPacksDep | |||
.filter(p => !stackPacksLib.stackPacksToInclude.find(p2 => p2.packId === p.id)) | |||
.map(p => p.id); | |||
expect(result).toMatchInlineSnapshot(` | |||
Array [ | |||
"joomla", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we convert this to toEqual([]);
with a comment that we shouldn't bother updating a stack pack until detectors are in place? :)
we needed these PRs GoogleChrome/lighthouse-stack-packs#59 GoogleChrome/lighthouse-stack-packs#60
and we needed to connect joomla detection to stack pack ...