Skip to content

PEP 1: Remove legacy reference to pep2pyramid.py - #626

Merged
ncoghlan merged 2 commits into
python:masterfrom
ncoghlan:pep-1-replace-legacy-pep2pyramid-reference
Apr 27, 2018
Merged

PEP 1: Remove legacy reference to pep2pyramid.py#626
ncoghlan merged 2 commits into
python:masterfrom
ncoghlan:pep-1-replace-legacy-pep2pyramid-reference

Conversation

@ncoghlan

Copy link
Copy Markdown
Contributor

This removes any reference to the technical details of online PEP
publication from PEP 1, replacing it with a reference to the
README in the PEP repository.

It also updates the README with relevant cross-references to the
pythondotorg project (based on some process pointers provided by
Mariatta Wijaya).

ResolvesGH-575.

This removes any reference to the technical details of online PEP
publication from PEP 1, replacing it with a reference to the
README in the PEP repository.
It also updates the README with relevant cross-references to the
pythondotorg project (based on some process pointers provided by
Mariatta Wijaya).
ResolvespythonGH-575.
@ncoghlan

Copy link
Copy Markdown
ContributorAuthor

@Mariatta@berkerpeksag I'm adjusting the references from the PEPs repo to the online publication process. Does what I've added look right from the pydotorg side?

@zwarezware 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, but I am not an expert on pythondotorg.

@ncoghlan
ncoghlan merged commit 499a7c2 into python:masterApr 27, 2018
@ncoghlan

Copy link
Copy Markdown
ContributorAuthor

I've gone ahead and merged it, since it's definitely less wrong than what was previously there.

If Berker or Mariatta spot any new problems in the updated text, then I can tackle them in a follow-up PR :)

@berkerpeksagberkerpeksag 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.

This looks great to me (and sorry for my late response.) Thanks!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@ncoghlan@berkerpeksag@zware@the-knights-who-say-ni