Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split the TRAILING_COMMA feature #763

Merged
merged 4 commits into from
Mar 25, 2019
Merged

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

@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