Skip to content

Conversation

@JelleZijlstra
Copy link
Collaborator

@JelleZijlstra JelleZijlstra commented Mar 15, 2019

It turns out that f(*x,) became legal in 3.5, but def f(*x,): only became legal in 3.6. Therefore, use two separate Features.

Fixes #759.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 972

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at ?%

Totals Coverage Status
Change from base Build 966: 0.0%
Covered Lines:
Relevant Lines: 0

💛 - Coveralls

@coveralls
Copy link

coveralls commented Mar 15, 2019

Pull Request Test Coverage Report for Build 979

  • 31 of 31 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 95.903%

Totals Coverage Status
Change from base Build 973: 0.01%
Covered Lines: 3020
Relevant Lines: 3149

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 972

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at ?%

Totals Coverage Status
Change from base Build 966: 0.0%
Covered Lines:
Relevant Lines: 0

💛 - Coveralls

We no longer add the trailing comma in 3.5.
@jxltom jxltom mentioned this pull request Mar 20, 2019
18 tasks
@maarcingebala
Copy link

Hey all, do you know when can we expect to have this PR merged and released? Issue #759 is currently blocking us from migrating to Black, as our project supports multiple Python version including Python 3.5. Thanks!

@JelleZijlstra JelleZijlstra merged commit cea13f4 into psf:master Mar 25, 2019
@JelleZijlstra JelleZijlstra deleted the trailcomma branch March 25, 2019 15:22
@maarcingebala
Copy link

@JelleZijlstra @zsol Thanks for taking care of this PR! Do you know when can we expect a new release?

@zsol
Copy link
Collaborator

zsol commented Mar 26, 2019

@maarcingebala sometime in April we're going to make the first stable release.

@maarcingebala
Copy link

Thanks for the info!

lirsacc added a commit to lirsacc/py-gql that referenced this pull request Apr 4, 2019
Until this black PR (psf/black#763) is released
the formatting adds spurious trailing commas on Python 3.5 which creates
a syntax error.
lirsacc added a commit to lirsacc/py-gql that referenced this pull request Apr 4, 2019
Until this black PR (psf/black#763)
is released the formatting adds spurious trailing commas on
Python 3.5 which creates a syntax error.
lirsacc added a commit to lirsacc/py-gql that referenced this pull request Apr 4, 2019
Until this black PR (psf/black#763)
is released the formatting adds spurious trailing commas on
Python 3.5 which creates a syntax error.
lirsacc added a commit to lirsacc/py-gql that referenced this pull request Apr 4, 2019
Until this black PR (psf/black#763)
is released the formatting adds spurious trailing commas on
Python 3.5 which creates a syntax error.
@roo-oliv
Copy link

roo-oliv commented Apr 30, 2019

Any updates on the stable release? This bug blocks our team for migrating to Black too.

bdraco added a commit to bdraco/python-zeroconf that referenced this pull request May 27, 2021
- The issue that prevented black from being used on this
  block was resolved in psf/black#763
anthonymedinawork added a commit to anthonymedinawork/py-gql that referenced this pull request Jan 24, 2024
Until this black PR (psf/black#763)
is released the formatting adds spurious trailing commas on
Python 3.5 which creates a syntax error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

black adds trailing comma despite using --target-version py35

5 participants