Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add validatorVersion on API responses #78

Merged
merged 1 commit into from
Aug 22, 2022
Merged

Conversation

PierrickP
Copy link
Collaborator

Return git hash on netlify, based on COMMIT_REF env variable or package.json's version
Close #77

@netlify
Copy link

netlify bot commented Aug 19, 2022

Deploy Preview for gbfs-validator ready!

Name Link
🔨 Latest commit 31c5325
🔍 Latest deploy log https://app.netlify.com/sites/gbfs-validator/deploys/62ff94dac179a600095ddf2c
😎 Deploy Preview https://deploy-preview-78--gbfs-validator.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 settings.

@PierrickP PierrickP force-pushed the add-validator-version branch 2 times, most recently from d4bf771 to e541382 Compare August 19, 2022 13:44
Return git hash on netlify, based on COMMIT_REF env variable or package.json's version
Close #77
@PierrickP PierrickP force-pushed the add-validator-version branch from e541382 to 31c5325 Compare August 19, 2022 13:49
@PierrickP PierrickP requested a review from isabelle-dr August 19, 2022 13:50
@PierrickP
Copy link
Collaborator Author

@AntoineAugusti is it ok for you ? You can test the PR version on https://deploy-preview-78--gbfs-validator.netlify.app/

@AntoineAugusti
Copy link
Contributor

image

Works well, thanks for working on this! 🙏

@thbar
Copy link

thbar commented Aug 19, 2022

Thank you @PierrickP!

@netlify
Copy link

netlify bot commented Aug 19, 2022

Deploy Preview for competent-payne-690ca9 failed.

Name Link
🔨 Latest commit 31c5325
🔍 Latest deploy log https://app.netlify.com/sites/competent-payne-690ca9/deploys/62ff94da92061b00081c00d9

@netlify
Copy link

netlify bot commented Aug 19, 2022

Deploy Preview for unruffled-hugle-914373 ready!

Name Link
🔨 Latest commit 31c5325
🔍 Latest deploy log https://app.netlify.com/sites/unruffled-hugle-914373/deploys/62ff94da78f29f00089d4661
😎 Deploy Preview https://deploy-preview-78--unruffled-hugle-914373.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 settings.

@netlify
Copy link

netlify bot commented Aug 19, 2022

Deploy Preview for kind-pike-a3f3f8 failed.

Name Link
🔨 Latest commit 31c5325
🔍 Latest deploy log https://app.netlify.com/sites/kind-pike-a3f3f8/deploys/62ff94da5d7055000832b482

@netlify
Copy link

netlify bot commented Aug 19, 2022

Deploy Preview for wizardly-engelbart-5c48ca failed.

Name Link
🔨 Latest commit 31c5325
🔍 Latest deploy log https://app.netlify.com/sites/wizardly-engelbart-5c48ca/deploys/62ff94dafd43b0000853d5b7

@netlify
Copy link

netlify bot commented Aug 19, 2022

Deploy Preview for wizardly-lichterman-770f54 failed.

Name Link
🔨 Latest commit 31c5325
🔍 Latest deploy log https://app.netlify.com/sites/wizardly-lichterman-770f54/deploys/62ff94dae727d000081d0144

Copy link
Contributor

@isabelle-dr isabelle-dr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thank you @PierrickP for working on this! Merging this PR

@isabelle-dr isabelle-dr merged commit 69b148e into master Aug 22, 2022
@AntoineAugusti
Copy link
Contributor

Wonderful, cheers! 🙇

josee-sabourin pushed a commit that referenced this pull request Apr 25, 2023
…170e

f43170e V3.0-RC Schemas  (#79)
680df70 Update pr-to-project.yml
c2c30a2 Typo fix in geofencing_zones.json (#78)
c9be765 Create pr-to-project.yml
0a17875 Create issues-to-project.yml
30e5a3a Update README.md

git-subtree-dir: gbfs-validator/versions/schemas
git-subtree-split: f43170e95d0969922a200a512cc5ab494702f79c
tdelmas added a commit to tdelmas/gbfs-validator that referenced this pull request Jun 28, 2023
…058d

22b058d fix: incorrect spec for "languages" property in system_information.json (MobilityData#90)
e9d891b Refer to later versions in gbfs_versions.json (MobilityData#87)
fbd3ced Add v3.0 to v3.0-RC schemas (MobilityData#88)
ef0d9a6 Fix schemas for v3.0-RC (MobilityData#85)
2b6ad1e FIX: replaces name field with localized string (MobilityData#82)
abe71f2 Fix invalid json, no trailing commas (MobilityData#83)
f43170e V3.0-RC Schemas  (MobilityData#79)
680df70 Update pr-to-project.yml
c2c30a2 Typo fix in geofencing_zones.json (MobilityData#78)
c9be765 Create pr-to-project.yml
0a17875 Create issues-to-project.yml
30e5a3a Update README.md

git-subtree-dir: gbfs-validator/versions/schemas
git-subtree-split: 22b058dc4a358baf451e006cde74775075929b3d
richfab pushed a commit that referenced this pull request Jun 29, 2023
* Git ignore for schemas subtree

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* Squashed 'gbfs-validator/versions/schemas/' changes from 5273f96..22b058d

22b058d fix: incorrect spec for "languages" property in system_information.json (#90)
e9d891b Refer to later versions in gbfs_versions.json (#87)
fbd3ced Add v3.0 to v3.0-RC schemas (#88)
ef0d9a6 Fix schemas for v3.0-RC (#85)
2b6ad1e FIX: replaces name field with localized string (#82)
abe71f2 Fix invalid json, no trailing commas (#83)
f43170e V3.0-RC Schemas  (#79)
680df70 Update pr-to-project.yml
c2c30a2 Typo fix in geofencing_zones.json (#78)
c9be765 Create pr-to-project.yml
0a17875 Create issues-to-project.yml
30e5a3a Update README.md

git-subtree-dir: gbfs-validator/versions/schemas
git-subtree-split: 22b058dc4a358baf451e006cde74775075929b3d

* remove dot files in schemas

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* v3-RC validation

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* Remove unused file

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* Add `serverOpts` required in Windows

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* Fix typo `recommended`

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* update test for 3.0-RC , 3.0 and upcoming minor versions

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* fix after rebase

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* Recommended.

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>

* Squashed 'gbfs-validator/versions/schemas/' changes from 22b058d..5f12243

5f12243 Patch manifest.json (#91)

git-subtree-dir: gbfs-validator/versions/schemas
git-subtree-split: 5f12243d5735a608eef697212a627b9090eea31a

---------

Signed-off-by: Tom <1955774+tdelmas@users.noreply.github.com>
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.

Output gbfs-validator version in the HTTP API response
4 participants