phillco / talon-axkit

Talon macOS accessibility magic!
MIT License
45 stars 12 forks source link

pre-commit: use shed to format Python opinionatedly #10

Closed phillco closed 2 years ago

phillco commented 2 years ago

Taking a good suggestion from https://github.com/knausj85/knausj_talon/issues/666, let's set up shed to reformat our code in an opinionated fashion. It also sorts imports and removes unused imports, which was my main motivation. :)

This doesn't set up any enforcement yet -- I need to learn more about how to use GitHub Actions (will do that later). For now, let's just make sure we run pre-commit run at some point during development, or use pre-commit install to install the commit hook.

Also if we forget I'll just reformat and pushed directly to main/experimental, it's not a big deal. Just want to get alignment on the actual style.

This just adds the config to reduce merge conflicts between the two branches -- to see what it looks like in action, see https://github.com/phillco/talon_axkit/pull/11.

phillco commented 2 years ago

Merging based on @nriley's signoff in Slack, we can adjust if we don't like it