Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Support code highlighting#1577

Merged
JelleZijlstra merged 5 commits into
python:masterpython/peps:masterfrom
ofek:highlightofek/peps:highlightCopy head branch name to clipboard
Nov 23, 2021
Merged

Support code highlighting#1577
JelleZijlstra merged 5 commits into
python:masterpython/peps:masterfrom
ofek:highlightofek/peps:highlightCopy head branch name to clipboard

Conversation

@ofek

@ofek ofek commented Aug 27, 2020

Copy link
Copy Markdown
Contributor

See: #1571 (comment)

Fine to try it in another PR, but looks like all that really is needed is to install pygments:

  1. The script step calls make to turn plaintext PEPs into HTML, and I assume pygments is only needed here to add colour to the generated HTML files (I'm afk this week so cannot verify this assumption.now).
  2. Then the deploy step calls deploy.bash which does make package to tar.gz the plaintext and generated files and images, and then calls aws to actually deploy the tar.gz to AWS.

@ofek ofek mentioned this pull request Aug 27, 2020
Merged
@brettcannon

brettcannon commented Aug 27, 2020

Copy link
Copy Markdown
Member

Have you checked if this will work with the deployment of the PEPs on python.org?

@ofek

ofek commented Aug 27, 2020

Copy link
Copy Markdown
Contributor Author

How would one test that?

@Mariatta

Mariatta commented Aug 28, 2020

Copy link
Copy Markdown
Member

The process to build the PEPs from python.org is described here https://pythondotorg.readthedocs.io/pep_generation.html

@ofek

ofek commented Aug 28, 2020

Copy link
Copy Markdown
Contributor Author 
django.db.utils.OperationalError: could not connect to server: No such file or directory
        Is the server running locally and accepting
        connections on Unix domain socket "/var/run/postgresql/.s.PGSQL.5432"?

@ofek

ofek commented Aug 28, 2020

Copy link
Copy Markdown
Contributor Author

I'm having difficulty building the site

@ofek

ofek commented Sep 2, 2020

Copy link
Copy Markdown
Contributor Author

Could one of you please try?

@brettcannon

brettcannon commented Sep 2, 2020

Copy link
Copy Markdown
Member

We don't 'have any experience with the web site, so it would be best to ask on the pythondotorg repo instead of here.

@hugovk

hugovk commented Sep 2, 2020

Copy link
Copy Markdown
Member

I was also unsuccessful, I tried both the Vagrant and manual methods from https://pythondotorg.readthedocs.io/install.html

@Mariatta

Mariatta commented Sep 4, 2020

Copy link
Copy Markdown
Member

Please report issue wrt pythondotorg guide in the GitHub.com/Python/pythondotorg repo

@hugovk hugovk mentioned this pull request Sep 6, 2020
Closed
@ofek ofek mentioned this pull request Sep 9, 2020
Closed
@ofek

ofek commented Sep 9, 2020

Copy link
Copy Markdown
Contributor Author

python/pythondotorg#1633

@EpicWink EpicWink mentioned this pull request Sep 10, 2020
Merged
2 tasks
@ofek

ofek commented Sep 18, 2020

Copy link
Copy Markdown
Contributor Author

@brettcannon Seems to work python/pythondotorg#1633 (comment)

@hugovk hugovk mentioned this pull request Sep 21, 2020
Closed
@ofek ofek requested a review from a team as a code owner June 2, 2021 02:34
@ofek

ofek commented Jun 2, 2021

Copy link
Copy Markdown
Contributor Author

I rebased, should be good to merge now

Mariatta
Mariatta reviewed Jun 18, 2021
View reviewed changes
Comment thread .github/workflows/build.yml Outdated
run: |
python -m pip install -U pip
python -m pip install -U docutils
python -m pip install -U docutils pygments

@Mariatta Mariatta Jun 18, 2021

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.

Perhaps we should create requirements.txt and pin the versions ofdocutils and pygments.

@ofek ofek Jun 19, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done!

@AA-Turner AA-Turner Jun 19, 2021

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.

Perils of reading commit info from emails and not checking the latest! Feel free to bump the docutils/sphix reqs, they are current as of April 2020, so 0.17.z / 4.y.z may be more appropriate.

A

@ofek ofek Jun 19, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Pushed commit!

AA-Turner
AA-Turner reviewed Jun 19, 2021
View reviewed changes
Comment thread requirements.txt
Pygments ~= 2.9.0
Sphinx ~= 4.0.2
docutils ~= 0.17.1

@AA-Turner AA-Turner Jun 19, 2021 •
edited
Loading

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.

Only question I'd have is if better to pin to compatible releases or all future? (~= vs >=) - historically been unpinned so I think that means the latest release has always been used.

(but I am not best placed to make this call, as no idea if e.g. previously there have been breaking changes to PEP rendering due to dependency changes, or the risk appetite for this in the future vs new features that may or may not be used)

A

@ofek ofek Jun 19, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay, changed it back

@ofek

ofek commented Jun 21, 2021

Copy link
Copy Markdown
Contributor Author

all good to merge?

@ofek

ofek commented Jun 23, 2021

Copy link
Copy Markdown
Contributor Author

is there any other concern here?

@brettcannon

brettcannon commented Jun 25, 2021

Copy link
Copy Markdown
Member

@ofek matter of time to look at the PR (which I have), deciding if we want it, and then watching python.org/dev/peps to make sure nothing breaks.

@ofek

ofek commented Nov 21, 2021

Copy link
Copy Markdown
Contributor Author

Any update on this?

@brettcannon

brettcannon commented Nov 23, 2021

Copy link
Copy Markdown
Member

None from me, but I'm not the only PEP editor. I suspect we are all just busy with other stuff at the moment.

@JelleZijlstra

JelleZijlstra commented Nov 23, 2021

Copy link
Copy Markdown
Member

Since it's been so long I'm going to re-trigger CI just to make sure. If it's still good I'm for merging this though.

@JelleZijlstra JelleZijlstra reopened this Nov 23, 2021
@JelleZijlstra JelleZijlstra merged commit 58629bc into python:master Nov 23, 2021
@ofek ofek deleted the highlight branch November 24, 2021 00:10
@erlend-aasland erlend-aasland mentioned this pull request May 13, 2022
Closed
@erlend-aasland erlend-aasland mentioned this pull request Jun 27, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mariatta Mariatta Mariatta left review comments

@AA-Turner AA-Turner AA-Turner left review comments

@brettcannon brettcannon brettcannon approved these changes

Assignees

No one assigned

Labels

CLA signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@ofek @brettcannon @Mariatta @hugovk @JelleZijlstra @AA-Turner @the-knights-who-say-ni
Morty Proxy This is a proxified and sanitized view of the page, visit original site.