pagure

#4425 Run black on all, included the tests and adjust the style tests
Merged by pingou. Opened by pingou.
black_on_all  into  master

Download 4425.patch
no initial comment

Should I remove the black fix commit from my PR then?

@lenkaseg we can yes.

To be honest, I'm a bit confused on why you've had to do them since black seems to be happy on the other PR and you haven't touched those files :s

No idea, but the test_style failed both here and when running on my computer.

rebased onto 646a34eaf95ec0a6fd76f6313d49b4306b914f91

rebased onto d5380fc7d0a949ddbedd5cfb0352b6ab86a10efe

2 new commits added

  • Project wide black fixes, including tests, docs and all
  • Adjust the style test to include the tests themselves

2 new commits added

  • Project wide black fixes, including tests, docs and all
  • Adjust the style test to include the tests themselves

2 new commits added

  • Project wide black fixes, including tests, docs and all
  • Adjust the style test to include the tests themselves

2 new commits added

  • Project wide black fixes, including tests, docs and all
  • Adjust the style test to include the tests themselves

2 new commits added

  • Project wide black fixes, including tests, docs and all
  • Adjust the style test to include the tests themselves

rebased onto 8f8cbbb149f2f21de389c69f619dea494ce2196b

2 new commits added

  • Project wide black fixes, including tests, docs and all
  • Adjust the style test to include the tests themselves

:thumbsup:

:thumbsup:

Thanks for the review :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.