Skip to content

Split specifications page into multiple pages - #386

Merged
ncoghlan merged 4 commits into
pypa:masterfrom
di:add-core-metadata-page
Oct 3, 2017
Merged

Split specifications page into multiple pages#386
ncoghlan merged 4 commits into
pypa:masterfrom
di:add-core-metadata-page

Conversation

@di

@di di commented Sep 29, 2017

Copy link
Copy Markdown
Member

This should fix #319:

  • Each section on /specifications is now it's own page;
  • Included the contents of PEP 345 in /specifications/core-metadata;
  • Added relevant versionadded directives to metadata field specs.

@pradyunsg

Copy link
Copy Markdown
Member

Included the contents of PEP 345 in /specifications/core-metadata

Would this result in redundant information?

@di

di commented Oct 1, 2017

Copy link
Copy Markdown
Member Author

@pradyunsg Yes, this will "duplicate" fields in PEP 345 for a short time until we can write a new PEP which supersedes it.

You might have missed that this was specifically requested in #319 (comment).

@pradyunsg

Copy link
Copy Markdown
Member

Yes, this will "duplicate" fields in PEP 345 for a short time until we can write a new PEP which supersedes it.

Oh, cool then. :)

You might have missed that this was specifically requested

Indeed.

@pradyunsg pradyunsg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

Still on the border about the core-metadata though. If it's till the next PEP comes, then cool. :)

@ncoghlan ncoghlan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for this! It generally looks good to me, just some requested tweaks to the extracted core metadata page.

(I'm aware that external section heading links to the specifications page will now just get the spec subsection listing, but I figure navigating from there will still be easy enough, so I'm not worried about it)

The source distribution format (``sdist``) is not currently formally defined.
Instead, its format is implicitly defined by the behaviour of the
standard library's ``distutils`` module when executing the ``setup.py sdist``
command.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Given PEP 517, this section isn't quite accurate any more, but fixing that is out of scope for this PR. I've filed #388 to keep track of it.

Comment thread source/specifications/core-metadata.rst Outdated

However, the version specifiers and environment markers sections of that PEP
have been superceded as described below. In addition, metadata files are
permitted to contain the following additional fields:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Given the change to include all the existing metadata fields as well, this intro paragraph need to be updated accordingly.

I'd also include an explicit note saying that:

  1. All fields added to later versions are considered optional in earlier versions (so metadata 1.3 fields can be listed even when the nominal metadata version is only 1.2)
  2. Metadata version 1.3 will be defined in a forthcoming PEP that also makes this URL the canonical source for the latest metadata description, so the fields listed as being added in 1.3 aren't typos, they're already-defined fields that will be included without further changes in the next version of the metadata specification

Comment thread source/specifications/core-metadata.rst Outdated

.. versionadded:: 1.0

Version of the file format; "1.2" is the only legal value.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Now that this is a multi-version format description, this should be reworded more in line with the description in PEP 416: https://www.python.org/dev/peps/pep-0426/#metadata-version

Currently permitted field values are then "1.0", "1.1", and "1.2"

@di

di commented Oct 2, 2017

Copy link
Copy Markdown
Member Author

@ncoghlan Thanks for your feedback! I think I've addressed everything.

@ncoghlan ncoghlan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@ncoghlan
ncoghlan merged commit 6c1cc86 into pypa:master Oct 3, 2017
@di
di deleted the add-core-metadata-page branch April 22, 2021 15:57
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.

Specifications page is hard to read

3 participants