Skip to content

Get rid of Tox - #1849

Merged
dvarrazzo merged 2 commits into
psycopg:masterfrom
edgarrmondragon:tox-4
Sep 23, 2026
Merged

dvarrazzo merged 2 commits into
psycopg:masterfrom
edgarrmondragon:tox-4

Conversation

@edgarrmondragon

@edgarrmondragon edgarrmondragon commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Simple migration

Signed-off-by: Edgar Ramírez Mondragón <edgarrm358@gmail.com>
@edgarrmondragon
edgarrmondragon marked this pull request as ready for review September 10, 2026 01:26
@dvarrazzo

Copy link
Copy Markdown
Member

I'd be tempted to kill tox tbh. What's the point of using it anymore? It's just an extra complication and Psycopg 3 benefited of getting rid of it.

@edgarrmondragon

Copy link
Copy Markdown
Contributor Author

I'd be tempted to kill tox tbh. What's the point of using it anymore? It's just an extra complication and Psycopg 3 benefited of getting rid of it.

Yeah, you've got a pretty simple Tox config here. Want me to take a stab at getting rid of it and updating the workflow accordingly?

@dvarrazzo

Copy link
Copy Markdown
Member

Why not. It's not an urgent maintenance issue but if you fancy "detoxing" the repo I'm not opposed to it.

@edgarrmondragon
edgarrmondragon marked this pull request as draft September 13, 2026 02:15
@edgarrmondragon edgarrmondragon changed the title Use Tox v4 Get rid of Tox Sep 13, 2026
@edgarrmondragon
edgarrmondragon marked this pull request as ready for review September 13, 2026 02:17
@dvarrazzo
dvarrazzo merged commit 7d03754 into psycopg:master Sep 23, 2026
12 checks passed
@dvarrazzo

Copy link
Copy Markdown
Member

Thank you very much for the cleanup! 💛

@edgarrmondragon
edgarrmondragon deleted the tox-4 branch September 23, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants