Add to documentation for commit instructions: YAPF

pgh-public-meetings / city-scrapers-pitt

Pittsburgh City Scrapers: sourcing public meetings in Pittsburgh

https://pgh-public-meetings.github.io/events/

MIT License

19 stars 66 forks source link

Add to documentation for commit instructions: YAPF #73

Closed wsnavely closed 4 years ago

wsnavely commented 4 years ago

Currently we hit YAPF issues in travis occasionally. We might want to add some documentation to help developers run YAPF before they check in, and fix any formatting problems.

To show YAPF discrepancies we can run: yapf --diff -r .

After doing this, developers can either apply the changes manually, or automatically apply them with: yapf -r -i .

wsnavely commented 4 years ago

Sorry, I'm not sure how to remove the "New Spider" label. This isn't a new spider.

ben-nathanson commented 4 years ago

I will add this to the new documentation

ben-nathanson commented 4 years ago

This is resolved in #84. See the file 04-development.md.