FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Travis CI: Matrix -> jobs and Python 3.6 to 3.8 by cclauss · Pull Request #1608 · RustPython/RustPython · GitHub

  1. RustPython
  2. RustPython

Repository navigation

Travis CI: Matrix -> jobs and Python 3.6 to 3.8 - #1608

Merged
youknowone merged 5 commits into
RustPython:masterfrom
cclauss:patch-2
Jan 12, 2020
ConversationCommits (5)ChecksFiles changed
Merged

Travis CI: Matrix -> jobs and Python 3.6 to 3.8#1608
youknowone merged 5 commits into
RustPython:masterfrom
cclauss:patch-2

Conversation

cclauss commented Nov 24, 2019 •
edited
Loading

Copy link
Copy Markdown
Contributor

https://docs.python.org/3/whatsnew/3.8.html#other-language-changes

For integers, the three-argument form of the pow() function now permits the exponent to be negative in the case where the base is relatively prime to the modulus. It then computes a modular inverse to the base when the exponent is -1, and a suitable power of that inverse for other negative exponents.

coolreader18 commented Nov 24, 2019

Copy link
Copy Markdown
Member

Hi @cclauss, RustPython is actually targeting Python ~3.6, hence the CPython versions for our tests being at 3.6. We might change the target version in the future, but for now I think we're good at 3.6.

youknowone commented Dec 4, 2019

Copy link
Copy Markdown
Member

Once we start to target specific python version seriously, fixing a target version will be helpful to step over to goal.

For now, because RustPython is not spending any cost to keep 3.6 compatibility (comparing to 3.7+) at this stage, I think we can just move to later version of CPython if CI doesn't fail. It seems just a preference problem at this stage.

windelbouwman commented Jan 12, 2020

Copy link
Copy Markdown
Contributor

I think we should aim for 3.6 ~ 3.8 ish behavior of python. I agree that as long as CI does not fail, it is okay to move onwards.

youknowone merged commit 947c4fe into RustPython:master Jan 12, 2020
cclauss deleted the patch-2 branch January 12, 2020 16:51
youknowone mentioned this pull request Jan 25, 2020
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

youknowone youknowone approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL