Merge ~jslarraz/review-tools:schema-split-definitions into review-tools:master
Proposed by
Jorge Sancho Larraz
Status: | Work in progress |
---|---|
Proposed branch: | ~jslarraz/review-tools:schema-split-definitions |
Merge into: | review-tools:master |
Diff against target: |
97 lines (+28/-15) 4 files modified
bin/store-query (+4/-10) reviewtools/schemas/definitions.json (+9/-0) reviewtools/schemas/snap.json (+9/-2) reviewtools/tests/schemas/test_schema_snap.py (+6/-3) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alex Murray | Approve | ||
Review via email:
|
Commit message
rt/schemas,
To post a comment you must log in.
Unmerged commits
- 46940fd... by Jorge Sancho Larraz
-
test:0 (build) coverage:0 (build) 1 → 2 of 2 results First • Previous • Next • Last
Stripping some value definitions from the snap.yaml schema will allow us to reuse them in the project (architectures, types, etc. in snap.yaml and snapcraft.yaml) or among projects (title, summary, etc. for snaps, rocks, etc.)
Reference to documents in local filesystem does not work on focal. This approach would require to move to a newer release or to place schemas in a publicly accessible location under a stable url pattern or ideally both.
Thoughts about it?