| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
@sbillinge Can we merge this? |
Sorry, something went wrong.
|
thanks @cadenmyers13 Next, please copy the .flake8 and black configs (this latter is in pyproject.toml, but don't copy over the whole thing, just the [tool.black] part. create a file with just this in if it is not there) and the pre-commit-config.yaml and start working on the flake8 fixes. Please split them into a few PRs so I am reviewing 4 or 5 files at a time. Periodically run the tests to make sure we didn't break anything! Thanks so much. |
Sorry, something went wrong.
|
@sbillinge Do you want me to create a pyproject.toml file with just the [tool.black] section in it? If so, should i remove setup.py? |
Sorry, something went wrong.
Good question. Not yet. If there is currently no pyproject.toml file, just create one and paste into it the [tool.black] section but nothing else. |
Sorry, something went wrong.
Got it. I already made the first PR with flake8 edits. I copied over the .flake8 file in that PR so the sooner we can get that merged the more I can split up the edits into different PRs, unless there is a better way to do this? |
Sorry, something went wrong.
I will try and merge it, but the quickest way to get it merged is to put in a PR with just the flake8 file.... |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.