PR Tests

[UKChris-osm]

Does GitHub Actions no longer perform a test when someone submits a PR? I've noticed that the more recent PR's don't include the green tick or red cross next to any of the PR commits.

When merging #11985 I didn't noticed the inclusion of the .geojson file extension for the country of Ireland when it wasn't required, and feel like I would have noticed this had the PR test flagged this up as a failed test.

[LaoshuBaby]

Does GitHub Actions no longer perform a test when someone submits a PR?

Looks like it still work.

When I merge #12019 with 1d34818 commit, a automatically test was called.

[LaoshuBaby]

Oh I haven't noticed this, actually the new commit haven't been checked. 😔

[UKChris-osm]

Yes it's during a PR that the test no longer appears to run - during a merge it seems to run just fine.

As another example, the recently merged #11924 has two matching displayName tags, casing a build error, which I think would have been noticed prior to merging if the PR test was still run.

@bhousel was the PR test removed when you moved over to bun?

[UKChris-osm]

Ah, it seems PR tests were removed in #11706 because of some issues with the new PR build Action?

[bhousel]

@bhousel was the PR test removed when you moved over to bun?

IIRC all the github action stuff came after the switch to bun.