Skip to content

Conversation

@dsch
Copy link

@dsch dsch commented Mar 30, 2022

fix for #66

The gitlab-ci schema was remove from schemastore repository and replaced by link to the file in gitlab repository.

I'm not sure how to handle the licensing of the schema file. Gitlab license states that client-side JavaScript (as far as I understand ci.json is part of the editor) are licensed under "MIT Expat" license.

@dsch dsch force-pushed the fix-gitlab-ci-schema branch from d5e6b9d to 034ba19 Compare March 30, 2022 09:45
@sirosen
Copy link
Member

sirosen commented Mar 30, 2022

Oh, wow! Thanks for this, and that was an embarrassing mistake to have in the repo.

I'm going to merge this, and I'll credit you in the changelog + add the license in a few minutes. I've been including the original licenses with some notes in the vendored schemas dir readme whenever pulling a schema from a new/distinct source.

Clearly I also need to fix the vendoring script to treat that sort of response as an error.


You can expect a release with this fix by end of day today. I'll get it done shortly.

@sirosen sirosen merged commit 37d502f into python-jsonschema:main Mar 30, 2022
@dsch dsch deleted the fix-gitlab-ci-schema branch April 7, 2022 10:14
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.

2 participants