Skip to content

Commit

Permalink
check for $defs first, since it is more future-oriented
Browse files Browse the repository at this point in the history
  • Loading branch information
rpatters1 authored and pboettch committed Jan 22, 2025
1 parent 1bcfdf8 commit 8edd521
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/json-validator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1396,8 +1396,8 @@ std::shared_ptr<schema> schema::make(json &schema,
}
return false;
};
if (!findDefinitions("definitions")) {
findDefinitions("$defs");
if (!findDefinitions("$defs")) {
findDefinitions("definitions");
}

attr = schema.find("$ref");
Expand Down

1 comment on commit 8edd521

@packit-as-a-service
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on your Packit configuration the settings of the lecris/nightly Copr project would need to be updated as follows:

field old value new value
additional_repos [] ['copr://@scikit-build/release']

Packit was unable to update the settings above as it is missing admin permissions on the lecris/nightly Copr project.

To fix this you can do one of the following:

  • Grant Packit admin permissions on the lecris/nightly Copr project on the permissions page.
  • Change the above Copr project settings manually on the settings page to match the Packit configuration.
  • Update the Packit configuration to match the Copr project settings.

Please retrigger the build, once the issue above is fixed.

Please sign in to comment.