search

gauteh / lieer

Fast email-fetching, sending, and two-way tag synchronization between notmuch and GMail
http://lieer.gaute.vetsj.com
Other
553 stars 62 forks source link

github action for black formatter and running black on the repo #247

Closed digitalsignalperson closed 1 year ago

digitalsignalperson commented 1 year ago

black.yml is setup per the official instructions here https://black.readthedocs.io/en/stable/integrations/github_actions.html

I tested these 3 commits here https://github.com/digitalsignalperson/lieer/pull/1 to see the action fail initially, then pass after running black . on the repo

black .
reformatted /home/andy1/git/lieer/lieer/__init__.py
reformatted /home/andy1/git/lieer/tests/__init__.py
reformatted /home/andy1/git/lieer/tests/test_local.py
reformatted /home/andy1/git/lieer/setup.py
reformatted /home/andy1/git/lieer/lieer/nobar.py
reformatted /home/andy1/git/lieer/lieer/resume.py
reformatted /home/andy1/git/lieer/lieer/local.py
reformatted /home/andy1/git/lieer/lieer/remote.py
reformatted /home/andy1/git/lieer/lieer/gmailieer.py

All done! ✨ 🍰 ✨
digitalsignalperson commented 1 year ago

Hmm, maybe it would help to split into 2 PRs, one just for the github action, and then one just for pushing the formatting changes from running black

gauteh commented 1 year ago

Thanks. I don't think I can merge this before #242 goes in, otherwise that PR is going to be completely messed up. It should be easy to update this PR later by keeping the first two commits and redoing the auto-formatting.

gauteh commented 1 year ago

Thanks, finally merged.