snazy commented on code in PR #1625: URL: https://github.com/apache/polaris/pull/1625#discussion_r2104281975
########## CONTRIBUTING.md: ########## @@ -64,32 +64,60 @@ Before starting on a large code change, please describe the concept/design of wh The best way to provide changes is to fork Apache Polaris repository on GitHub and provide a Pull Request with your changes. To make it easy to apply your changes please use the following conventions: -* Every Pull Request should have a matching GitHub Issue. -* Create a branch that will house your change: - -```bash -git clone https://github.com/apache/polaris -cd polaris -git fetch --all -git checkout -b my-branch origin/main -``` - - Don't forget to periodically rebase your branch: - -```bash -git pull --rebase -git push GitHubUser my-branch --force -``` - - Ensure the code is properly formatted: - -```bash -./gradlew format -``` +#### Before opening a pull request * Pull Requests should be based on the `main` branch. +* Create a branch that will house your change: + ```bash + git clone https://github.com/apache/polaris + cd polaris + git checkout main + git pull + git checkout -b my-branch + ``` +* Work on the changes of your pull requests locally. +* Ensure the code is properly formatted: + ```bash + ./gradlew format + ``` +* Ensure the code is passing the tests: + ```bash + ./gradlew check + ``` +* You may want to push your changes to your personal Polaris fork. Git will emit a URL that you can use to create the pull request. Do not create the pull request yet. + ```bash + git push --set-upstream your-github-accout + ``` + +#### Opening a pull request + +* Before opening a pull request, squash all commits in your branch into a single commit Review Comment: It's meant as a "tip" - moved the controversial one to a separated "Tips" section. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
