Skip to content

Conversation

@vfdev-5
Copy link
Collaborator

@vfdev-5 vfdev-5 commented Dec 18, 2021

Related to #2366

Description:

  • Updated black to 21.12b0
  • FIxed modifications for other folders and ignite/contrib

Check list:

  • New tests are added (if a new feature is added)
  • New doc strings: description and/or example code are in RST format
  • Documentation is updated (if required)

@vfdev-5 vfdev-5 merged commit ee762fb into pytorch:black-21.12b0 Dec 18, 2021
@vfdev-5 vfdev-5 deleted the update-black-to-2112b0-others branch December 18, 2021 16:17
vfdev-5 added a commit that referenced this pull request Dec 21, 2021
* Updated black to 21.12b0 and fixed codebase for others folders (#2367)

* Updated black to 21.12b0 and fixed codebase for others folders

* Updated black

* formatting fixed (#2372)

* fix: remove tailing commas that would mislead black formating

* autopep8 fix

* fix: left out tailing comma

* chg: change black version

* fix: test_deprecated assert failed with additional newline

* fix: typo

* fix: fix func_no_reasons test assert failed

Co-authored-by: louis-she <louis-she@users.noreply.github.com>

Co-authored-by: vfdev <vfdev.5@gmail.com>
Co-authored-by: louis-she <louis-she@users.noreply.github.com>
Ishan-Kumar2 pushed a commit to Ishan-Kumar2/ignite that referenced this pull request Dec 26, 2021
* Updated black to 21.12b0 and fixed codebase for others folders (pytorch#2367)

* Updated black to 21.12b0 and fixed codebase for others folders

* Updated black

* formatting fixed (pytorch#2372)

* fix: remove tailing commas that would mislead black formating

* autopep8 fix

* fix: left out tailing comma

* chg: change black version

* fix: test_deprecated assert failed with additional newline

* fix: typo

* fix: fix func_no_reasons test assert failed

Co-authored-by: louis-she <louis-she@users.noreply.github.com>

Co-authored-by: vfdev <vfdev.5@gmail.com>
Co-authored-by: louis-she <louis-she@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant