@jsenv/file-size-impact 12.1.7 → 12.1.8
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +2 -2
- package/src/internal/formatComment.js +20 -10
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@jsenv/file-size-impact",
|
|
3
|
-
"version": "12.1.
|
|
3
|
+
"version": "12.1.8",
|
|
4
4
|
"description": "Add files size impact into pull requests",
|
|
5
5
|
"license": "MIT",
|
|
6
6
|
"repository": {
|
|
@@ -9,7 +9,7 @@
|
|
|
9
9
|
"directory": "packages/jsenv-file-size-impact"
|
|
10
10
|
},
|
|
11
11
|
"engines": {
|
|
12
|
-
"node": ">=
|
|
12
|
+
"node": ">=16.13.0"
|
|
13
13
|
},
|
|
14
14
|
"publishConfig": {
|
|
15
15
|
"access": "public",
|
|
@@ -90,11 +90,12 @@ const renderCommentBody = ({
|
|
|
90
90
|
: "none"
|
|
91
91
|
const meta = event === "deleted" ? beforeMerge.meta : afterMerge.meta
|
|
92
92
|
|
|
93
|
-
const { sizeMapBeforeMerge, sizeMapAfterMerge, sizeImpactMap } =
|
|
94
|
-
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
93
|
+
const { sizeMapBeforeMerge, sizeMapAfterMerge, sizeImpactMap } =
|
|
94
|
+
getSizeMapsOneFile({
|
|
95
|
+
sizeNames: transformationKeys,
|
|
96
|
+
beforeMerge,
|
|
97
|
+
afterMerge,
|
|
98
|
+
})
|
|
98
99
|
const { showSizeImpact, formatFileRelativeUrl } = metaToData(meta, {
|
|
99
100
|
fileRelativeUrl,
|
|
100
101
|
sizeBeforeMerge: sizeMapBeforeMerge[0],
|
|
@@ -103,9 +104,13 @@ const renderCommentBody = ({
|
|
|
103
104
|
})
|
|
104
105
|
fileByFileImpact[fileRelativeUrl] = {
|
|
105
106
|
event,
|
|
106
|
-
manifestKeyBeforeMerge: beforeMerge
|
|
107
|
+
manifestKeyBeforeMerge: beforeMerge
|
|
108
|
+
? beforeMerge.manifestKey
|
|
109
|
+
: undefined,
|
|
107
110
|
manifestKeyAfterMerge: afterMerge ? afterMerge.manifestKey : undefined,
|
|
108
|
-
relativeUrlBeforeMerge: beforeMerge
|
|
111
|
+
relativeUrlBeforeMerge: beforeMerge
|
|
112
|
+
? beforeMerge.relativeUrl
|
|
113
|
+
: undefined,
|
|
109
114
|
relativeUrlAfterMerge: afterMerge ? afterMerge.relativeUrl : undefined,
|
|
110
115
|
sizeMapBeforeMerge,
|
|
111
116
|
sizeMapAfterMerge,
|
|
@@ -186,10 +191,13 @@ ${JSON.stringify(groupConfig, null, " ")}
|
|
|
186
191
|
}
|
|
187
192
|
|
|
188
193
|
const formulateMergeImpact = ({ pullRequestBase, pullRequestHead }) => {
|
|
189
|
-
return `<p>
|
|
194
|
+
return `<p>Impact on file sizes when merging <em>${pullRequestHead}</em> into <em>${pullRequestBase}</em>.</p>`
|
|
190
195
|
}
|
|
191
196
|
|
|
192
|
-
const metaToData = (
|
|
197
|
+
const metaToData = (
|
|
198
|
+
meta,
|
|
199
|
+
{ fileRelativeUrl, sizeBeforeMerge, sizeAfterMerge, sizeImpactMap },
|
|
200
|
+
) => {
|
|
193
201
|
if (typeof meta === "boolean") {
|
|
194
202
|
return {
|
|
195
203
|
showSizeImpact: true,
|
|
@@ -210,7 +218,9 @@ const metaToData = (meta, { fileRelativeUrl, sizeBeforeMerge, sizeAfterMerge, si
|
|
|
210
218
|
}
|
|
211
219
|
}
|
|
212
220
|
|
|
213
|
-
console.warn(
|
|
221
|
+
console.warn(
|
|
222
|
+
`meta must be a boolean or an object, received ${meta} for ${fileRelativeUrl}`,
|
|
223
|
+
)
|
|
214
224
|
return {
|
|
215
225
|
showSizeImpact: Boolean(meta),
|
|
216
226
|
}
|