Skip to main content

Contributing

We ❀️ every form of contribution. The following document aims to provide enough context to work with our codebase and to open pull requests that follow our conventions. If this document does not provide enough help, open a new issue and we'll gladly help you get started.

Work with the code#

Airy Core uses Bazel to build and test itself. We suggest you to install bazelisk, a small utility that will install the right version of Bazel for you.

Build#

You can build the whole platform using the following command:

bazel build //...

and build a specific project like so:

bazel build //backend/api/communication:all

Test#

To run tests, execute the following command:

bazel test //...

You can also run tests of a specific project like so:

bazel test //backend/conversations/api/communication:all

Lint#

You can lint the entire code base by executing the ./scripts/lint.sh script.

We use language specific linters:

Java, buildifier, and prettier are run as test targets for each package, so you can run:

bazel test //my/package:checkstyle
bazel test //my/package:prettier
bazel test //my/package:buildifier

You can also run:

bazel run //:fix

to try fixing issues automatically (not supported for checkstyle).

Managing Java dependencies#

If you add, remove, or change a dependency from the repositories.bzl, you must re-pin dependencies using the following command:

REPIN=1 bazel run @unpinned_maven//:pin

Exploring the code base#

Bazel has an extensive query language you can use to dig deeper into projects. Here are a few examples:

# show all deps of a given project
bazel query "deps(//backend/api/conversations:all)" --output label
# show the available tests of a given project
bazel query "tests(//backend/api/conversations:all)" --output label
# show all the packages under a specific path
bazel query "backend/..." --output package

If you are not familiar with a specific project, you can also run the following query:

bazel query "backend/..."

The query shows all the targets produced under that specified package. It can help getting started.

Bazel also offers a friendly and powerful autocompletion, please refer to this document to install it locally.

Publishing and running images locally#

You can test your images locally at runtime if you have a local instance of Airy Core running in Minikube.

Publish image to minikube's registry#

In order for the Airy Core cluster to have access to a newly built image, you need to publish the image to Minikube's (built-in) docker registry. Do this by pointing your shell's docker environment to the docker instance in minikube:

eval $(minikube -p airy-core docker-env)

Next, create an image with bazel, which will automatically load the image into Minikube's registry. For example:

bazel run //backend/api/communication:image

You can verify this action with docker images and you can restore the shell environment to point to your local docker instance with eval $(minikube -p airy-core docker-env -u).

Run image on Airy Core#

Next, you will want to run this image in the Airy Core cluster. For this example that means you need to:

kubectl patch deployment api-communication -p '{"spec":{"template":{"spec":{"containers":[{"name":"app","image":"bazel/backend/api/communication:image","imagePullPolicy":"Never"}]}}}}'

Once this is done, the image will be up and running in your local cluster.

Naming conventions#

In order to organize our releases in the best possible way, we follow a few critical conventions.

Branches#

Branches must abide to the following format:

<branch-type>/<issue-number>-<description>

branch-type is defined as follows:

  • feature or feat are used for feature branches
  • bug, fix, hotfix are used for bug fixes
  • enhancement is used for improvements of existing features
  • doc or docs are used for documentation changes
  • chore is used for maintenance tasks on the repo

The description field must use kebab case.

Given these conventions here are a few examples:

feat/42-the-meaning-of-life
bug/24-say-the-vat-is-good
hotfix/4242-til-json-is-not-a-subset-of-js
enhancement/3333-speed-up-airy-instance-creation

Commits#

To keep a clean track of what is being released, every feature must contain only one commit when merged. The commit message of the squashed commit is very important, since it will be used to keep track of the features in a release.

The conventional format is: [#issue] Description (Description must start with a capital letter). For the example, if your pull request refers to the issue "Introduce meaning of life" with number 42, the squashed commit message must be:

[#42] Introduce meaning of life
Fixes #42
Last updated on