Skip to content

Commit ac6892c

Browse files
authored
use object key for links + table formatting
1 parent f2d5882 commit ac6892c

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

.github/workflows/lighthouse-ci.yml

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -61,25 +61,26 @@ jobs:
6161
'| --- | --- | --- | --- | --- | --- |',
6262
];
6363
64-
Object.values(links).forEach(link => {
65-
const relevantManifests = manifests.filter(manifest => manifest.url === link);
64+
Object.entries(links).forEach(([pageUrl, reportUrl]) => {
65+
const relevantManifests = manifests.filter(manifest => manifest.url === pageUrl);
6666
const results = relevantManifests.map(manifest => manifest.summary);
6767
const averagedResults = {};
68-
68+
6969
if (results.length > 0) {
7070
Object.keys(results[0]).forEach(key => {
7171
averagedResults[key] = formatResult(
7272
results.reduce((acc, cur) => acc + cur[key], 0) / results.length
7373
);
7474
});
75-
75+
7676
const score = res => res >= 90 ? '🟢' : res >= 50 ? '🟠' : '🔴';
77-
77+
const urlForTable = pageUrl.includes('/en/') ? pageUrl.substring(pageUrl.indexOf('/en/')) : pageUrl;
78+
7879
comment.push(
79-
`| [Link](${link}) | ${score(averagedResults.performance)} ${averagedResults.performance} | ${score(averagedResults.accessibility)} ${averagedResults.accessibility} | ${score(averagedResults['best-practices'])} ${averagedResults['best-practices']} | ${score(averagedResults.seo)} ${averagedResults.seo} | ${score(averagedResults.pwa)} ${averagedResults.pwa} |`
80+
`| [${urlForTable}](${reportUrl}) | ${score(averagedResults.performance)} ${averagedResults.performance} | ${score(averagedResults.accessibility)} ${averagedResults.accessibility} | ${score(averagedResults['best-practices'])} ${averagedResults['best-practices']} | ${score(averagedResults.seo)} ${averagedResults.seo} | ${score(averagedResults.pwa)} ${averagedResults.pwa} |`
8081
);
8182
} else {
82-
console.error('No results found for URL:', link);
83+
console.error('No results found for URL:', pageUrl);
8384
}
8485
});
8586

0 commit comments

Comments
 (0)