Skip to content

Conversation

pyrooka
Copy link
Member

@pyrooka pyrooka commented Nov 16, 2022

PR summary

This commit adds the Black code formatter to the project
and updates the Makefile to use it. There are 2 targets that run the black tool:

  • make lint: this will check the code with both pylint and black
  • make lint-fix: this will reformat the code using black.

It also re-formats the Python file by running make lint-fix.

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
@pyrooka pyrooka force-pushed the nb/add-black-formatter branch 2 times, most recently from c42a87b to ad8833c Compare November 17, 2022 14:53
Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>

This reverts commit ad8833c.
@pyrooka pyrooka force-pushed the nb/add-black-formatter branch from a855a31 to 23b9905 Compare November 18, 2022 09:43
This reverts commit 23b9905.

Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
@pyrooka
Copy link
Member Author

pyrooka commented Nov 18, 2022

I ended up choosing the lazy way (revertception?) to fix the builds.

@pyrooka pyrooka requested a review from padamstx November 18, 2022 10:03
@padamstx
Copy link
Contributor

I ended up choosing the lazy way (revertception?) to fix the builds.

What is a "revertception"? 😂

Copy link
Contributor

@padamstx padamstx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pyrooka pyrooka merged commit 390add1 into main Nov 18, 2022
@pyrooka pyrooka deleted the nb/add-black-formatter branch November 18, 2022 14:58
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 0.29.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants