Skip to content

Extend release validation to support v1 API files #42

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

Merged
merged 1 commit into from
Mar 25, 2025
Merged

Conversation

theory
Copy link
Member

@theory theory commented Mar 25, 2025

The API Server adds additional properties to the Meta API, an which here is called Release (because it contains release metadata). This caused validation to fail for metadata fetched from api.pgxn.org.

Extend the validation of Release metadata to optionally include these properties. Define a new v1 schema, api-extras.schema.json, and add it to release.schema.json. Like the existing schemas, it applies strict validation for the new properties.

Since the new schemas validate file paths, copy path.schema.json from the v2 schemas and apply it to the other paths in the v1 schema.

Increment to v0.6.0 and prep for release.

@theory theory requested a review from vrmiguel March 25, 2025 20:27
@theory theory self-assigned this Mar 25, 2025
@theory theory force-pushed the api-properties branch 2 times, most recently from 8390531 to 5275c9b Compare March 25, 2025 21:18
The API Server adds additional properties to the [Meta API], an which
here is called Release (because it contains release metadata). This
caused validation to fail for metadata fetched from api.pgxn.org.

Extend the validation of Release metadata to optionally include these
properties. Define a new v1 schema, `api-extras.schema.json`, and add it
to `release.schema.json`. Like the existing schemas, it applies strict
validation for the new properties.

Since the new schemas validate file paths, copy `path.schema.json` from
the v2 schemas and apply it to the other paths in the v1 schema.

Increment to v0.6.0 and prep for release.

  [Meta API]: https://github.com/pgxn/pgxn-api/wiki/meta-api#api-server-structure
@theory theory merged commit e1563e1 into main Mar 25, 2025
114 of 126 checks passed
@theory theory deleted the api-properties branch March 25, 2025 23:45
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