Skip to content

py3: remove futures, six and encoding#1401

Merged
insspb merged 4 commits intomasterfrom
drop27-35
May 27, 2020
Merged

py3: remove futures, six and encoding#1401
insspb merged 4 commits intomasterfrom
drop27-35

Conversation

@insspb
Copy link
Copy Markdown
Member

@insspb insspb commented May 26, 2020

No description provided.

@insspb insspb changed the title Remove futures, six and encoding. Making code python 3 only. Making code python 3 only: Remove futures, six and encoding. May 26, 2020
@insspb insspb added feature This issue/PR relates to major feature request. code style This issue/PR relates to the project code style modification labels May 26, 2020
@insspb insspb self-assigned this May 26, 2020
@insspb insspb added the needs-review PR Only: This PR require review from other developer label May 26, 2020
@ssbarnea ssbarnea changed the title Making code python 3 only: Remove futures, six and encoding. py3: remove futures, six and encoding May 27, 2020
@qlty-cloud-legacy
Copy link
Copy Markdown

Code Climate has analyzed commit 3e34a3d and detected 5 issues on this pull request.

Here's the issue category breakdown:

Category Count
Security 5

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.2% change).

View more on Code Climate.

@ssbarnea
Copy link
Copy Markdown
Member

@insspb If you could split it by change-type it would be much easier to review and merge.

@insspb
Copy link
Copy Markdown
Member Author

insspb commented May 27, 2020

@ssbarnea It is already splitted. for #1401, #1402 and many tests-related PR that will be done one by one. :)

I will merge this one in minutes, as it is already approved by you ;)

@insspb insspb merged commit e8fbf92 into master May 27, 2020
@insspb insspb deleted the drop27-35 branch May 27, 2020 10:13
@insspb insspb added this to the 2.0.0 milestone May 27, 2020
@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 feature This issue/PR relates to major feature request. needs-review PR Only: This PR require review from other developer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants