From 55c91661bbbcbe0c27dce912a1e969e49ee1f8fc Mon Sep 17 00:00:00 2001 From: Roman Zavarnitsyn Date: Thu, 20 Feb 2025 17:05:53 +0100 Subject: [PATCH] chore(contributing): Update contributing guide (#4190) --- CONTRIBUTING.md | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 375f5cdc3e..8e2c8b78bf 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -27,6 +27,36 @@ To run the build and tests: make compile ``` +# Format + +To format the changed code and make CI happy you can run: + +```shell +make format +``` + +or + +```shell +./gradlew spotlessApply +``` + +# Binary compatibility validation + +To prevent breaking ABI changes and exposing things we should not, we make use of https://github.com/Kotlin/binary-compatibility-validator. If your change intended to introduce a new public method/property or modify the existing one you can overwrite the API declarations to make CI happy as follows (overwrites them from scratch): + +```shell +make api +``` + +or + +```shell +./gradlew apiDump +``` + +However, if your change did not intend to modify the public API, consider changing the method/property visibility or removing the change altogether. + # CI Build and tests are automatically run against branches and pull requests