Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Rename rst peps to .rst #39

Closed
wants to merge 2 commits into from

Conversation

Carreau
Copy link
Contributor

@Carreau Carreau commented Jun 27, 2016

Not sure how many things that would break when pushing on pythondotorg. That's pretty easy to redo if it's not the right time.

Carreau added 2 commits June 27, 2016 11:11
simply

```ipython
files = ! ls   *.txt

for name in files:
    with open(name) as f:
        if 'Content-Type: text/x-rst' in f.read():
            !git mv {name} {name[:-3]}rst
```
@berkerpeksag
Copy link
Member

Thanks! Yes, this will definitely break python.org :) I have a WIP branch that fixes a couple issues with the PEP importer. I can tweak it look for .rst files too but I'm not sure I can get to it this week.

@brettcannon brettcannon changed the title Rename rst peps to .rst [WIP] Rename rst peps to .rst Jul 1, 2016
@brettcannon
Copy link
Member

I've marked this PR as a work-in-progress until the python.org side is dealt with.

@Carreau
Copy link
Contributor Author

Carreau commented Jul 1, 2016

I've marked this PR as a work-in-progress until the python.org side is dealt with.

Sure, if you want a cleaner PR queue, feel free to close, we can reopen when time is right.
It's easy to redo.

@brettcannon
Copy link
Member

Is there an issue or PR for the pydotorg side of things that we can track in regards to this PR?

@berkerpeksag
Copy link
Member

No, I was planning to fix it (also hg.python.org links needs to be updated) while working on python/pythondotorg#825 :)

@abalkin
Copy link
Member

abalkin commented Aug 7, 2016

Note that PEP preambles are not in valid reST format. Would it break PEP processing tools if they are converted to reST comments (.. directive) or pre-formatted text (:: directive)?

@brettcannon
Copy link
Member

@abalkin yes, they would break. Eventually the headers would need to be changed to be valid reST, but one thing at a time. 😄

@Carreau
Copy link
Contributor Author

Carreau commented Sep 26, 2016

Closing to keep things clean, it's easy to redo.

@Carreau Carreau closed this Sep 26, 2016
lukpueh added a commit to lukpueh/peps that referenced this pull request Oct 24, 2019
Add myself (Joshua) to the authors list
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants