Conversation
|
Thanks for the PR! This section of the codebase is owned by @madskristensen and |
|
Thanks. I can merge once CI goes green ( |
c03affe to
514ac46
Compare
|
I check similar case
schemastore/src/api/json/catalog.json Line 6884 in b6a418e
https://github.com/SchemaStore/schemastore/tree/master/src/test Thus, should be OK to remove community schema |
|
I would like to confirm a few points of concern.
The advantage of using the official schema is that you automatically get the latest schema information. |
|
There are 35 Maybe contribute to official repo if description is not enough. Schemastore is using official docker compose schema. I don't know how It should be same. |
|
It would be better to delay the merge as the current official There is no clear ownership for who will contribute to the official schema. I will likely not be able to work on this issue for a while as my resources are diverted to another project 😓 You should probably think of |
|
Hi, @hyperupcall Has there ever been a similar situation for reference?
|
|
This PR is stale because it has been open 60 days with no activity. Comment or this will be closed in 7 days. |
|
This PR was closed because it has been stalled for 7 days with no activity. |
use official cargo schema
Related
Now upstream have official cargo schema
#5026
Notes
src/schemas/json/cargo.jsonand related test files are not yet removed