rethinkdb / rethinkdb-go

Go language driver for RethinkDB
Apache License 2.0
1.65k stars 182 forks source link

Upgrade to Python 3.8 #496

Closed CodyDWJones closed 2 years ago

CodyDWJones commented 3 years ago

Reason for the change If the ReQL test generator is run with Python 3.8 it fails for multiple reasons. Python 3.8 was released almost two years ago so it seems reasonable to make it a minimum requirement for this task.

Description The main change was switching from ast.Num to ast.Constant, since the ast package in 3.8 now provides all constants as ast.Constant instances.

I left alone the uses of ast.Str and others, which are deprecated but continue to work for now.

I regenerated the tests using the revised code, but committed this separately because there are many "spurious" changes. Somehow the database and table names used by the test definitions have changed. I was unable to find any uses of the old names in the RethinkDB repository.

Note: this includes the .travis.yml change from #494.

Code examples

Checklist

References