Move PR preview to Checks tab and add statistics

[ZeLonewolf]

This PR adds statistics to the checks tab, which will show the change in layer size.
Additionally, this moves the PR Preview comment into the checks tab, to avoid email spamming.

[jleedev]

Maybe add a comment in scripts/stats.js or in its output expressing in words that this is the length of the serialized style JSON.

[ZeLonewolf]

Maybe add a comment in scripts/stats.js or in its output expressing in words that this is the length of the serialized style JSON.

Added in 14d4403

[ZeLonewolf]

Sample output, from the github actions log:

Style size statistics

	main	this PR	change
Layers	349	349	0
Size (b)	1005817	1005817	0

Layer count comparison

	main	this PR	change
background	1	1	0
landuse	1	1	0
park	3	3	0
aeroway	7	7	0
landcover	2	2	0
boundary	9	9	0
water	3	3	0
waterway	3	3	0
transportation	299	299	0
building	1	1	0
transportation_name	3	3	0
poi	2	2	0
water_name	2	2	0
aerodrome_label	4	4	0
place	9	9	0

Layer size comparison

	main	this PR	change
background	122	122	0
landuse	309	309	0
park	2365	2365	0
aeroway	2059	2059	0
landcover	361	361	0
boundary	13691	13691	0
water	912	912	0
waterway	3543	3543	0
transportation	920221	920221	0
building	292	292	0
transportation_name	8838	8838	0
poi	10094	10094	0
water_name	4647	4647	0
aerodrome_label	5951	5951	0
place	32062	32062	0

[1ec5]

Not sure what’s going on with the random links unfortunately. Apparently something is getting piped through an environment where GitHub Actions is parsing content to autolink it.

[ZeLonewolf]

I manually copy/pasted the content from the output log of the github action, so that may or may not occur when the github action does the writing.

[ZeLonewolf]

From some testing, it's happening at the browser/paste level so probably ignorable.

[ZeLonewolf]

The random linking issue does not occur when writing directly to the Checks tab.

[github-actions]

PR Preview:

• Map
• Shield Test
• style.json
• shields.json
• taginfo.json

Sprite Sheets:

[1ec5]

I’m not seeing the statistics in the Checks tab. Is it dependent on merging to main?

[1ec5]

Never mind, I missed this in the Check tab.

[1ec5]

Sorry for the delay. I’ll be glad to see this information go in the Checks tab rather than spam the comments.