search

Duckapple / Kat

Kattis shell for getting examples, testing and submitting.
19 stars 6 forks source link

Defaults to "python" #54

Open Hjaltesorgenfrei opened 3 years ago

Hjaltesorgenfrei commented 3 years ago

The config is currently defaulting to python, with no clear difference between python 2 and 3. This is problematic as the kattis platform differs between the two, so it should be either python2 or python3

jlndk commented 3 years ago

Since python 2 has been deprecated we should default to python 3 IMO. This would be a non-breaking change since it would only take place on config generation (https://github.com/Duckapple/Kat/blob/4db376128aaa46c0d0a443e19c70f7dc8db3982a/helpers/config.py#L60)

Hjaltesorgenfrei commented 3 years ago

I think that is reasonable. But as long as Kattis allows python2, it should be an option, it still runs faster than python3