Skip to content

Wrong link to PEPs source file when the file ends in .rst #431

Closed

Description

@Naereen

Hi,
The issue is simple: the peps pages (eg pep-0553) include a link to the text source file, hosted on GitHub.
But the link points to non-existing .txt files, as some of them are stored as .rst !
The issue comes from the current template:

[<b><a href="%(pephome)s/pep-%(pepnum)s.txt">PEP Source</a></b>]

Activity

  1. added a commit that references this issue on Oct 18, 2017
  2. Naereen commented on Oct 18, 2017

    @Naereen
    ContributorAuthor

    #432 fixes this.
    (I don't see a reason this could be a bad idea, please someone review)

  3. Naereen commented on Oct 18, 2017

    @Naereen
    ContributorAuthor

    OK I just saw that half the PEPs are stored as txt and half are stored in rst...
    So my fix is stupid, but something should be fixed.

  4. changed the title [-]Wrong link to PEPs source .txt file on GitHub : they are .rst ![/-] [+]Wrong link to PEPs source .txt file on GitHub : some are .rst ![/+] on Oct 18, 2017
  5. brettcannon commented on Oct 18, 2017

    @brettcannon
    Member

    It's a known issue that's waiting for #4 to be completed.

  6. changed the title [-]Wrong link to PEPs source .txt file on GitHub : some are .rst ![/-] [+]Wrong link to PEPs source file when the file ends in .rst[/+] on Oct 18, 2017
  7. Naereen commented on Oct 18, 2017

    @Naereen
    ContributorAuthor

    OK, thanks.

  8. added a commit that references this issue on Nov 15, 2017
  9. Mariatta commented on Nov 15, 2017

    @Mariatta
    Member

    This PR should take care of it: python/pythondotorg#1195.

  10. Naereen commented on Nov 15, 2017

    @Naereen
    ContributorAuthor

    @Mariatta thanks! That's a fix on the website part, and I would be in favor of accepting it asap (maybe even in a more generic way, cf. my review/comment).

    As @brettcannon pointed it out, the important changes to be made are #4, known from a long time.

  11. Mariatta commented on Nov 15, 2017

    @Mariatta
    Member

    Thanks @Naereen :)
    Issue #4 has been completed, the content of the PEPs have all been converted to restructured text format.
    I think the consensus for now is to not rename the files from .txt to .rst since GitHub won't properly display the complete file history after the rename. See the discussion.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions