zeroshade commented on code in PR #124: URL: https://github.com/apache/arrow-adbc/pull/124#discussion_r971116704
########## CONTRIBUTING.md: ########## @@ -175,3 +172,57 @@ $ pytest -vvx ### Ruby The Ruby libraries are bindings around the GLib libraries. + +## Opening a Pull Request + +Before opening a pull request, please run the static checks, which are +enforced via [`pre-commit`](https://pre-commit.com/). This will run +linters, formatters, and other analysis. For example: + +```shell +$ pip install pre-commit +# Set up hooks +$ pre-commit install +# Run manually +$ pre-commit run +Check Xml............................................(no files to check)Skipped +Check Yaml...........................................(no files to check)Skipped +Fix End of Files.....................................(no files to check)Skipped +Trim Trailing Whitespace.............................(no files to check)Skipped +clang-format.........................................(no files to check)Skipped +cmake-format.........................................(no files to check)Skipped +cpplint..............................................(no files to check)Skipped +Google Java Formatter................................(no files to check)Skipped +black................................................(no files to check)Skipped +flake8...............................................(no files to check)Skipped +isort................................................(no files to check)Skipped +# Hooks automatically run on commit +$ git commit +``` + +When committing, please follow [Conventional +Commits][conventional-commits]. This helps maintain semantic +versioning of components. + +Please use the following commit types: `build`, `chore`, `ci`, `docs`, +`feat`, `fix`, `perf`, `refactor`, `revert`, `style`, `test`. + +Please use the following scopes: + +- `c/driver/postgres`, `java/driver-manager`, …: for a component and + all its bindings. For example, `c/driver-manager` covers the C/C++ + driver manager and its GLib and Python bindings, while + `java/driver/flight-sql` covers only the Flight SQL driver for Java. + (The scope names are derived from the filesystem paths.) +- `c/format`, `go/format`, `java/format`: for the core API definitions + (adbc.h for C/C++, adbc.go for Go, adbc-core for Java). + +For example: + +``` +feat(c/driver/postgres): implement prepared statements + +fix(java/driver/jdbc): adjust SQL type mapping for JDBC driver Review Comment: haha, well chosen :stuck_out_tongue: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
