ci: Add Python 3.8 to the CI Python matrix

[matthewfeickert]

Now that Python 3.8 has been added in the the setup-python GitHub Action, it should be added to the Python runtime CI matrix.

[gordonwatts]

Well, shoot. A number of things over took this, so I'm going to have to do some merging back to get this branch working correctly. And see what has changed in 3.8.

[gordonwatts]

Ok - the 3.8 contains some real changes to how the AST is laid out. So for now I'll leave this here, and I will have to work on this once we get this into ServiceX.

[matthewfeickert]

Ok - the 3.8 contains some real changes to how the AST is laid out. So for now I'll leave this here, and I will have to work on this once we get this into ServiceX.

Sounds good. This isn't needed, as you already merged master in, but to keep things easy to keep track of in the commit graph I'm just going to rebase this off of the main project's master.

[gordonwatts]

Closing this b.c. it is now very out of date. Due to ast changes, we are going to have to do some real work before moving to 3.8.

[matthewfeickert]

Closing this b.c. it is now very out of date. Due to ast changes, we are going to have to do some real work before moving to 3.8.

Sounds good. Good news is that it is a trivial rebase whenever you get there, so I won't delete the branch. :)

[gordonwatts]

Thanks!!