Skip to content

Python3: Changed io.open to build-in open (PEP3116)#1408

Merged
ssbarnea merged 1 commit intomasterfrom
io_open_vs_open
May 28, 2020
Merged

Python3: Changed io.open to build-in open (PEP3116)#1408
ssbarnea merged 1 commit intomasterfrom
io_open_vs_open

Conversation

@insspb
Copy link
Copy Markdown
Member

@insspb insspb commented May 28, 2020

Import of io module was required in python <3.
Since python 3 open is alias for io.open and direct import of io module not required.

@insspb insspb added the code style This issue/PR relates to the project code style modification label May 28, 2020
@insspb insspb added this to the 2.0.0 milestone May 28, 2020
@insspb insspb self-assigned this May 28, 2020
@qlty-cloud-legacy
Copy link
Copy Markdown

Code Climate has analyzed commit eedb6f8 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Security 2

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 99.7% (0.0% change).

View more on Code Climate.

@ssbarnea ssbarnea merged commit d8672b1 into master May 28, 2020
@ssbarnea ssbarnea deleted the io_open_vs_open branch May 28, 2020 15:29
@simobasso simobasso mentioned this pull request May 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code style This issue/PR relates to the project code style modification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants