Skip to content

Conversation

@marksmayo
Copy link

  • cleaned up imports
  • added 3, 3.5, 3.6+ trailing commas
  • tidied whitespace
  • formatted with black
 - cleaned up imports - added 3, 3.5, 3.6+ trailing commas - tidied whitespace - formatted with black
@paketb0te
Copy link
Contributor

+1 for using black for formatting

@initialcommit-io
Copy link
Contributor

Hi @marksmayo,

Thanks for this - I hadn't used black before and @paketb0te suggested we used it in a different issue they opened. I just looked into it and I think we should go for it.

But I've made some conflicting code changes since you submitted this PR. Can you pull down the updates and run your formatting again, and submit a new PR?

I'll merge that and then we can use black going forward when committing new code.

@paketb0te
Copy link
Contributor

-> See my Comment in #23

@initialcommit-io
Copy link
Contributor

Hi @marksmayo,

I took care of the formatting with black and committed.

If you'd like you can clean up the imports and add the trailing commas in a new PR. Not sure if the whitespace updates still apply after the reformatting with black has been done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants