Skip to content

Conversation

uniocto
Copy link
Contributor

@uniocto uniocto commented May 8, 2021

@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label May 8, 2021
@iritkatriel iritkatriel requested a review from bitdancer May 8, 2021 13:17
@iritkatriel
Copy link
Member

iritkatriel commented May 8, 2021

Change the title of the PR to something like:
bpo-24147: Document csv Dialect class defaults. Patch by Brandon Milam.

@uniocto uniocto changed the title bpo-24147: Apply Brandon Milam's patch about csv doc bpo-24147: Document csv Dialect class defaults. Patch by Brandon Milam. May 8, 2021
@uniocto uniocto changed the title bpo-24147: Document csv Dialect class defaults. Patch by Brandon Milam. bpo-24147: Document csv Dialect class defaults. Patch by Brandon Milam May 8, 2021
@github-actions
Copy link

github-actions bot commented Jun 8, 2021

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Jun 8, 2021
@MaxwellDupre
Copy link
Contributor

Doesn't compile:
fix-issue-24147 $…  make -C Doc/ html
make: Entering directory '/home/me/Documents/GitHub/cpython/Doc'
mkdir -p build
Building NEWS from Misc/NEWS.d with blurb
PATH=./venv/bin:$PATH sphinx-build -b html -d build/doctrees -W . build/html
Running Sphinx v4.5.0
loading pickled environment... done
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 487 source files that are out of date
updating environment: 0 added, 447 changed, 2 removed
reading sources... [100%] whatsnew/index
Warning, treated as error:
/home/me/Documents/GitHub/cpython/Doc/c-api/complex.rst:49:Error in declarator or parameters
Error in declarator or parameters
Invalid C declaration: Expected identifier, got user-defined keyword: complex. Remove it from c_extra_keywords to allow it as identifier.
Currently c_extra_keywords is ['alignas', 'alignof', 'bool', 'complex', 'imaginary', 'noreturn', 'static_assert', 'thread_local']. [error at 39]
Py_complex _Py_c_neg(Py_complex complex)
make: *** [Makefile:51: build] Error 2
make: Leaving directory '/home/me/Documents/GitHub/cpython/Doc'

The target is Python 2, would it be possible to re-target on the latest version?
The CSV Doc hasn't changed so still relevant.

@github-actions github-actions bot removed the stale Stale PR or inactive for long period of time. label Aug 7, 2022
@python-cla-bot
Copy link

python-cla-bot bot commented Apr 6, 2025

The following commit authors need to sign the Contributor License Agreement:

CLA signed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review docs Documentation in the Doc dir skip news
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

6 participants