Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • slapos slapos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Merge requests 122
    • Merge requests 122
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedi
  • slaposslapos
  • Merge requests
  • !1616

Merged
Created Jul 26, 2024 by Rafael Monnerat@rafael👻Owner

kvm/theia: Dont use allOf on resilient to extend properties

  • Overview 4
  • Commits 1
  • Changes 2

allOf implies that all (both) schemas should be valid, but additionalProperties false wont allow it, since it become multually invalid. (You can validate only one or None)

Just use $ref to expand the list, it end up with the expected outcome reducing complexity.

Note: kvm wasn't using additionalProperties false but it was changed because it makes it future proof, in case we set it (we should set).

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: resilient-schema-allof-fixup
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7