Skip to content

Conversation

@stm2
Copy link
Contributor

@stm2 stm2 commented Sep 15, 2023

Migrated to python3, mainly accept-orders.py. I have relatively low confidence in this and would like to see it tested more.

Copy link
Member

@ennorehling ennorehling left a comment

Choose a reason for hiding this comment

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

Ich habe ein paar Fragen, weil ich schon lange nichts mehr mit Python mache, aber im großen und ganzen sieht das vernünftig aus, und genau die Sorte Änderungen, die ich erwartet hatte. Testen ist schwierig, denn das einzige Environment in dem ich das ganze benutze, ist der Live Server. Sogar die Test-Auswertung am Sonntag benutzt die selbe Version der Skripte aus ~/eressea/server, was mir schon eine Weile ein Ärgernis ist, vielleicht sollte ich das mal deutlicher separieren.

@ennorehling
Copy link
Member

Oh, ist schon gemacht,. Prima!

@ennorehling ennorehling merged commit 6e06b28 into eressea:develop Sep 17, 2023
@stm2 stm2 deleted the python3 branch September 17, 2023 21:22
ennorehling added a commit that referenced this pull request Nov 27, 2023
* upgrade to python3

* get rid of python2 compatibility

---------

Co-authored-by: Enno Rehling <enno@eressea.de>
ennorehling added a commit that referenced this pull request Mar 2, 2024
* upgrade to python3

* get rid of python2 compatibility

---------

Co-authored-by: Enno Rehling <enno@eressea.de>
ennorehling added a commit that referenced this pull request Mar 3, 2024
* upgrade to python3

* get rid of python2 compatibility

---------

Co-authored-by: Enno Rehling <enno@eressea.de>
ennorehling added a commit that referenced this pull request Mar 9, 2024
* upgrade to python3

* get rid of python2 compatibility

---------

Co-authored-by: Enno Rehling <enno@eressea.de>
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