Skip to content

bpo-34533: Remove all static state from _csv module #16078

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.

Sign up for GitHub

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

Closed
eduardo-elizondo wants to merge 3 commits into from
+155 −144
Closed

bpo-34533: Remove all static state from _csv module #16078

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
60dd2d7
Make _csv compatible with PEP384
eduardo-elizondo Sep 13, 2019
caf5ecf
Fixes
eduardo-elizondo Sep 23, 2019
1d78d00
Update news
eduardo-elizondo Sep 23, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Make _csv compatible with PEP384
  • Loading branch information
@eduardo-elizondo
eduardo-elizondo committed Sep 13, 2019
commit 60dd2d7dd6ddc9fac75bde7d769a6c269c27be69
2 changes: 1 addition & 1 deletion Lib/test/test_csv.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -454,7 +454,7 @@ def test_copy(self):
def test_pickle(self):
for name in csv.list_dialects():
dialect = csv.get_dialect(name)
for proto in range(pickle.HIGHEST_PROTOCOL + 1):
for proto in range(2, pickle.HIGHEST_PROTOCOL + 1):
self.assertRaises(TypeError, pickle.dumps, dialect, proto)

class TestCsvBase(unittest.TestCase):
Expand Down
1 change: 1 addition & 0 deletions Misc/NEWS.d/next/C API/2019-09-13-02-26-42.bpo-34533.S13_VG.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Make the _csv module compatible with PEP384
Loading