Skip to content

Document the JSON schema of direct_url.json#1512

Merged
webknjaz merged 3 commits into
pypa:mainfrom
edgarrmondragon:direct_url.json-schema
Feb 29, 2024
Merged

Document the JSON schema of direct_url.json#1512
webknjaz merged 3 commits into
pypa:mainfrom
edgarrmondragon:direct_url.json-schema

Conversation

@edgarrmondragon
Copy link
Copy Markdown
Contributor

@edgarrmondragon edgarrmondragon commented Feb 28, 2024

Adds a JSON schema for the structure of direct_url.json.

Based on https://github.com/edgarrmondragon/pep610/blob/41a39253521637e85f364f60d1106c25dfa03d02/tests/fixtures/direct_url.schema.json with some changes to make it more accurate:

  • Updated the JSON schema draft version to 2019-09 so deprecated can be used for archive_info.hash.
  • Fixed the pattern of archive_info.hash to actually accomodate valid algorithm names. e.g. md5=... wouldn't have been valid before.

See the discuss thread for more context.

Relevant docs preview: https://python-packaging-user-guide--1512.org.readthedocs.build/en/1512/specifications/direct-url-data-structure/#json-schema


📚 Documentation preview 📚: https://python-packaging-user-guide--1512.org.readthedocs.build/en/1512/

@brettcannon
Copy link
Copy Markdown
Member

@edgarrmondragon FYI CI failed.

Comment thread source/specifications/direct-url-data-structure.rst
@edgarrmondragon

This comment was marked as resolved.

@brettcannon

This comment was marked as resolved.

@chrysle

This comment was marked as resolved.

@webknjaz webknjaz enabled auto-merge February 29, 2024 18:51
@webknjaz webknjaz added this pull request to the merge queue Feb 29, 2024
Merged via the queue into pypa:main with commit 9218401 Feb 29, 2024
@edgarrmondragon edgarrmondragon deleted the direct_url.json-schema branch February 29, 2024 18:56
edgarrmondragon added a commit to edgarrmondragon/pep610 that referenced this pull request Mar 1, 2024
Will merge whatever
pypa/packaging.python.org#1512 ends up with,
even if it's not accepted in the end.
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.

5 participants