Skip to content

Add render tests#477

Merged
wipfli merged 19 commits into
mainfrom
monaco
May 12, 2025
Merged

Add render tests#477
wipfli merged 19 commits into
mainfrom
monaco

Conversation

@wipfli
Copy link
Copy Markdown
Collaborator

@wipfli wipfli commented Apr 30, 2025

No description provided.

Comment thread tiles/monaco.sh Outdated
@wipfli wipfli marked this pull request as draft May 1, 2025 06:52
Comment thread styles/src/flavors.ts Outdated
@wipfli
Copy link
Copy Markdown
Collaborator Author

wipfli commented May 9, 2025

This now looks all ready:

image

I made a change in the styling to have a failing render test above. You can click on the render test ci and then go to the "Upload render tests results.html on failure" job. This brings you to a link where you can download a zip file containing the CI's render test results.html file. Download an open it and you will see this:

image

@wipfli wipfli changed the title Add Monaco build test Add render tests May 9, 2025
@wipfli wipfli marked this pull request as ready for review May 9, 2025 13:42
Comment thread render-tests/tests/roads/default/style.json
Comment thread render-tests/tests/roads/default/style.json
Comment thread render-tests/generate_pmtiles.sh
@bdon
Copy link
Copy Markdown
Member

bdon commented May 11, 2025

LGTM, can merge after reverting the intentional breakage for CI failure

@sonarqubecloud
Copy link
Copy Markdown

@wipfli wipfli merged commit 09c3b57 into main May 12, 2025
6 checks passed
@wipfli wipfli deleted the monaco branch May 12, 2025 05:10
@wipfli wipfli mentioned this pull request May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants