Skip to content

Google Cloud Run: Improve schema#4631

Merged
hyperupcall merged 17 commits intoSchemaStore:masterfrom
ewjoachim:cloud_run_improve
Apr 14, 2025
Merged

Google Cloud Run: Improve schema#4631
hyperupcall merged 17 commits intoSchemaStore:masterfrom
ewjoachim:cloud_run_improve

Conversation

@ewjoachim
Copy link
Contributor

closes #4625

Improves the Cloud Run schema by:

  • removing all unused objects
  • Adding "additionalProperties": false everywhere since Google enforces that (apparently)
  • searching descriptions for constraints and implementing them as jsonschema
  • removing unsupported properties
  • removing read-only properties

@github-actions
Copy link
Contributor

github-actions bot commented Apr 2, 2025

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

@ewjoachim ewjoachim force-pushed the cloud_run_improve branch from a2e06c2 to f8fd7d6 Compare April 2, 2025 22:01
@ewjoachim
Copy link
Contributor Author

Note: all the comments above are just suggestions. If we merge as-is (provided there are no review comments), I'll already be plenty satisfied.

@ewjoachim ewjoachim changed the title Remove unused elements Google Cloud Run: Improve schema Apr 2, 2025
@madskristensen
Copy link
Contributor

Let me know when this is ready to merge

@ewjoachim ewjoachim force-pushed the cloud_run_improve branch from f8fd7d6 to 5cfd506 Compare April 3, 2025 07:17
@ewjoachim
Copy link
Contributor Author

ewjoachim commented Apr 3, 2025

@madskristensen This is ready to merge

@hyperupcall
Copy link
Member

Thanks!

@hyperupcall hyperupcall merged commit 511d090 into SchemaStore:master Apr 14, 2025
4 checks passed
@ewjoachim
Copy link
Contributor Author

Thanks to you :)

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.

Improve the (existing) Google Cloud Run schema

3 participants