trinodb / trino-python-client

Python client for Trino
Apache License 2.0
311 stars 154 forks source link

README updates #226

Closed hashhar closed 1 year ago

hashhar commented 1 year ago

Most of this is "refactor".

Actual useful/impacting content changes are:

PTAL @ebyhr at Update releasing instructions in README specially. I'm not sure if the trailing slash was intentionally left out (to prevent copy and paste?).

hashhar commented 1 year ago

I left out pre-commit setup for now since CI enforces things for now but we can decide what parts to offload to PR authors (e.g. import sorting, automated reformat).

I'm wary of requiring something like mypy enforced on PR authors (it must be enforced in CI however) because not everyone is familiar with it and we want people to be able to open PRs and we can help with parts they aren't confident about. We can raise the bar over time.

hashhar commented 1 year ago

Thanks!. Re I left out pre-commit setup for now are you referring to #208 (comment) ?

Yes, I mean someone should add pre-commit setup instruction to README. I also want to look into whether it's possible to have "profiles" in pre-commit so that CI is more strict than local.

Or alternatively we can only add things we really want to check before commit to pre-commit (like sort imports, auto reformat) but keep other things like mypy and tests outside of it.

hashhar commented 1 year ago

I'll follow up with dedicated DEVELOPMENT.md/CONTRIBUTING.md.