cal-itp / mobility-marketplace

A one-stop resource for modern, equitable and accessible public transit products and services.
https://www.camobilitymarketplace.org/
GNU Affero General Public License v3.0
5 stars 1 forks source link

Provider Map Data Auto Update #546

Closed github-actions[bot] closed 3 months ago

github-actions[bot] commented 5 months ago

It's that time again! The warehouse has delivered new data for us to use. This is an automatic pull request created by the provider-map-jobs.yml workflow; it is triggered via a cron that runs every Sunday at midnight UTC.

netlify[bot] commented 5 months ago

Deploy Preview for cal-itp-mobility-marketplace ready!

Name Link
Latest commit c78c2d167fb7df12389a5e71a5994e817d02b971
Latest deploy log https://app.netlify.com/sites/cal-itp-mobility-marketplace/deploys/6611e782cd52e3000854d47a
Deploy Preview https://deploy-preview-546--cal-itp-mobility-marketplace.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 5 months ago

Deploy Preview for cal-itp-mobility-marketplace ready!

Name Link
Latest commit bd2d0860630585c5e0ddea89cb46e22f94c05da9
Latest deploy log https://app.netlify.com/sites/cal-itp-mobility-marketplace/deploys/665bbba533f16d0008ae072e
Deploy Preview https://deploy-preview-546--cal-itp-mobility-marketplace.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

allejo commented 5 months ago

Yeah, let me investigate; it looks like this was triggered because the GeoJSON generation gets saved without a newline. So that's "different," and then our formatter is running as a separate commit after the PR is opened and adding that newline. In the end, they cancel each other out, and you end up with this PR.

My thinking is to run the formatting as part of the GeoJSON commit process to avoid this.

allejo commented 3 months ago

@thekaveman this is on you to approve and merge 😄