Contributing to NativeLink

NativeLink welcomes contribution from everyone. Here are the guidelines if you are thinking of helping us:

Contributions

Contributions to NativeLink or its dependencies should be made in the form of GitHub pull requests. Each pull request will be reviewed by a core contributor (someone with permission to land patches) and either landed in the main tree or given feedback for changes that would be required. All contributions should follow this format, even those from core contributors.

Should you wish to work on an issue, please claim it first by commenting on the GitHub issue that you want to work on it. This is to prevent duplicated efforts from contributors on the same issue.

Git setup

NativeLink has a somewhat specific contribution process to ensure consistent quality across all commits. If anything in the following guide is unclear to you please raise an issue so that we can clarify this document.

In your GitHub settings, set up distinct authentication and signing keys. For further information see:
- https://docs.github.com/en/authentication/connecting-to-github-with-ssh/generating-a-new-ssh-key-and-adding-it-to-the-ssh-agent
- https://docs.github.com/en/authentication/connecting-to-github-with-ssh/adding-a-new-ssh-key-to-your-github-account
Fork the TraceMachina/nativelink repository by clicking on the Fork button.

Clone your fork:

git clone git@github.com:yourusername/nativelink

Warning

Don't use

git clone git@github.com:TraceMachina/nativelink

cd into the cloned repository:
```
cd nativelink
```
Add TraceMachina/nativelink as a remote repository and call it upstream:
```
git remote add upstream git@github.com:TraceMachina/nativelink
```

Tip

To verify whether the setup was successful, run git remote -v. The output should look like this:

origin   git@github.com:yourusername/nativelink (fetch)
origin   git@github.com:yourusername/nativelink (push)
upstream     git@github.com:TraceMachina/nativelink (fetch)
upstream     git@github.com:TraceMachina/nativelink (push)

Finally, configure git to sign your commits with the keys you set up previously. Create a .gitconfig file in your home directory like so:

[user]
     name = Your Full Name
     email = the-email-you-use-on-github@example.com
     signingkey = ~/.ssh/your_private_signing_key

[gpg]
     format = ssh  # Or gpg if you use a GPG key.

[commit]
     gpgsign = true

[tag]
     gpgsign = true

Local development setup

NativeLink ships almost all of its tooling in a nix flake which is configured via a flake.nix file in the root of the repository. While it's possible to work on some parts of the codebase without this environment, it'll make your life much easier since it lets you reproduce most of CI locally.

Install Nix with flakes: https://github.com/NixOS/experimental-nix-installer For further information on Nix Flakes see: https://nixos.wiki/wiki/Flakes.

Optionally (but highly recommended), install direnv and hook it into your shell:

nix profile install nixpkgs#direnv

# Add the right hook for your shell: https://direnv.net/docs/hook.html

# Restart your terminal.

# When you `cd` into the nativelink repository, you should see a message
# asking you to run `direnv allow`. Do this and you're good to go.

Note

If you don't want to use direnv, you'll need to enter the flake manually with nix develop every time you enter the nativelink directory, switch branches or make changes to the nix files.

Tip

To verify that the environment is active, run env | grep NIX. You should see several *NIX_* environment variables.

The environment doesn't ship a full C++ toolchain yet. Install a recent version of Clang manually:

# Use your distros preferred package manager (pacman, emerge, apt etc), or
# install via nix:
nix profile install nixpkgs#clang

Common workflows

These are some common workflows that you'll encounter during development on NativeLink.

Creating pull requests

NativeLink doesn't allow direct commits or human-created side branches in the TraceMachina/nativelink repository. This holds for contributors in the TraceMachina organization. To create a pull request:

Ensure that your personal fork is up-to-date with upstream:

git switch main  # Switch to your local main branch
git pull -r upstream main  # Fetch upstream and rebase your current branch
                           # against upstream's main branch
git push  # Push your updated main to your own fork

Tip

Use git log to check that your branches are in order:

# A `git log` should look like this:
commit ... (HEAD -> main, upstream/main, origin/main, origin/HEAD)
...

Create a new branch for the change you want to make:
```
git switch -c some-feature
```
After making changes to the source code create a commit with git commit. To keep commits and the git history uniform and readable keep the following rules in mind:
- Use a capital letter to start the commit and use an imperative tone for the title.
- Don't end the title with a period.
- Keep the first line as short as possible. If your feature is complex, add additional information in the commit message body.
- If you feel like you need the word and in the commit title, the commit might try to do too many things at once and you should consider splitting it into separate commits.
- The commit message body should have a maximum line length of 72 characters. This is to keep the git log readable with raw terminals.
```
# Good.
Add some feature

# Bad - trailing period
Add some feature.

# Bad - not imperative
Adds some feature

# Bad - details should be in the body
Add some complex feature and try to put all info in the title

# Bad - commit should be split
Add some feature and actually some other feature as well
```
Push your commit with git push. This will prompt you to set a remote branch for the commit:
```
git push --set-upstream origin some-feature
```
Go to https://github.com/TraceMachina/nativelink/pulls where you should see a button that you can click to create to create a new pull request from your fork to the main repository.
Once you opened the pull request, click on the purple Reviewable button in the GitHub page for the pull request to add reviewers with +@somereviewer.

The reviewers will take it from there and guide you through any potential remaining issues. Feel free to ask for help if you have trouble getting CI for your pull request green.
If you need to make additional changes, don't use a regular git commit on the pull request branch. Instead use git commit --amend and git push -f to update the commit in-place. The changes between the commit versions will remain visible in the Reviewable UI.

Using `git rebase`

When you start working on a feature your git log looks something like this:

gitGraph
    commit id: "aaa"
    branch origin/main
    branch origin/some-complex-feature
    commit id: "Add some complex feature"

Loading

For complex features your commit might become outdated over time:

gitGraph
    commit id: "aaa"
    branch origin/main
    branch origin/some-complex-feature
    commit id: "Add some complex feature"
    checkout main
    commit id: "bbb"
    commit id: "ccc"

Loading

To get up-to-date with the latest branch, get the latest upstream commit and rebase your branch onto the new main branch:

git switch main                 # Switch to the origin/main branch
git pull -r upstream main       # Sync the local main branch with upstream/main
git push                        # Push the new local main to origin/main
git switch some-complex-feature # Go back to the feature branch
git rebase main                 # Rebase the feature branch onto the local main
git push -f                     # Update the PR on GitHub

After this the history will be fine again:

gitGraph
    commit id: "aaa"
    commit id: "bbb"
    commit id: "ccc"
    branch origin/main
    branch origin/some-complex-feature
    commit id: "Add some complex feature"

Loading

Fixing rust formatting

When working on Rust code bazel test commands automatically run rustfmt on all source files. If you get errors from these checks, run the rustfmt Bazel target to format the sources.

For Bash/Zsh:

bazel run \
  --@rules_rust//:rustfmt.toml=//:.rustfmt.toml \
  @rules_rust//:rustfmt

For Windows PowerShell:

bazel run `
  --@rules_rust//:rustfmt.toml=//:.rustfmt.toml `
  @rules_rust//:rustfmt

Running pre-commit hooks

Ensure you're in the Nix development environment as described in the Local Development Setup. To run the hooks:

pre-commit run -a

This will automatically apply some fixes like automated line fixes and format changes. Note that changed files aren't automatically staged. Use git add to add the changed files manually to the staging area.

Setting up rust-analyzer

rust-analyzer works reasonably well out of the box due to picking up the manifest for the nativelink crate, but it isn't integrated with Bazel by default. In order to generate a project configuration for rust-analyzer, run the @rules_rust//tools/rust_analyzer:gen_rust_project target:

bazel run @rules_rust//tools/rust_analyzer:gen_rust_project

This will generate a rust-project.json file in the root directory. This file needs to be regenerated every time new files or dependencies are added in order to stay up-to-date. You can configure rust-analyzer can pick it up by setting the rust-analyzer.linkedProjects configuration option.

If you use VS Code, you can configure the following tasks.json file to automatically generate this file when you open the editor:

{
  "version": "2.0.0",
  "tasks": [
    {
      "label": "Generate rust-project.json",
      "command": "bazel",
      "args": ["run", "@rules_rust//tools/rust_analyzer:gen_rust_project"],
      "options": {
        "cwd": "${workspaceFolder}"
      },
      "group": "build",
      "problemMatcher": [],
      "presentation": {
        "reveal": "never",
        "panel": "dedicated"
      },
      "runOptions": {
        "runOn": "folderOpen"
      },
      "dependsOn": "Build nativelink"
    },
    {
      "label": "Build nativelink",
      "command": "bazel",
      "args": ["build", "//:nativelink"],
      "options": {
        "cwd": "${workspaceFolder}"
      },
      "group": "build",
      "presentation": {
        "reveal": "silent",
        "panel": "shared"
      }
    }
  ]
}

And to configure rust-analyzer to use that project, set this in your .vscode/settings.json:

{
     "rust-analyzer.linkedProjects": ["rust-project.json"]
}

Generating documentation

Automatically generated documentation is still under construction. To view the documentation for the nativelink-* crates, run the docs command in the nix flake:

docs

To build individual crate-level docs:

# All docs
bazel build docs

# A single crate
bazel build nativelink-config:docs

To run documentation tests with Bazel:

bazel test doctests

Writing documentation

NativeLink largely follows the Microsoft Style Guide.

NativeLink implements its documentation style guide via Vale. The pre-commit hooks forbid errors but permit warnings and suggestions. To view all of Vale's suggestions invoke it directly:

vale somefile

Creating releases

To keep the release process in line with best practices for open source repositories, not all steps are automated. Specifically, tags should be signed and pushed manually and the release notes should be human readable beyond what most automatically generated changelogs provide.

Bump the current version in the following files:
- MODULE.bazel
- Cargo.toml
- nativelink-*/Cargo.toml
Run git cliff --tag=0.x.y > CHANGELOG.md to update the changelog. You might need to make manual adjustments to cliff.toml if git-cliff doesn't put a commit in the right subsection.
Create the commit and PR. Call it Release NativeLink v0.x.y.
Once the PR is merged, update your local repository and origin:
```
git switch main
git pull -r upstream main
git push
```
Create a signed tag on the release commit and give it the same tag message as the name of the tag. This tag should be the version number with a v prefix:
```
git tag -s v0.x.y

# tag message should be: v0.x.y
```
Push the signed tag to the origin repository:
```
git push origin v0.x.y
```
Pushing the tag triggers an additional GHA workflow which should create the container images in your own fork. Check that this workflow is functional. If the CI job in your fork passes, push the tag to upstream:
```
git push upstream v0.x.y
```
The images for the release are now being created. Go to the Tags tab in GitHub and double-check that the tag has a green Verified marker next to it. If it does, select Create a release from tag and create release notes. You can use previous release notes as template by clicking on the "Edit" button on a previous release and copy-pasting the contents into the new release notes.

Make sure to include migration instructions for all breaking changes.

Explicitly list whatever changes you think are worth mentioning as Major changes. This is a fairly free-form section that doesn't have any explicit requirements other than being a best-effort summary of notable changes.
Once all notes are in line, click Publish Release.

Conduct

NativeLink Code of Conduct is available in the CODE_OF_CONDUCT file.

Generating code coverage

You can generate branch-based coverage reports via:

nix run .#nativelinkCoverageForHost

The result symlink contains a webpage with the visualized report.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CONTRIBUTING.md

CONTRIBUTING.md

Contributing to NativeLink

Contributions

Git setup

Local development setup

Common workflows

Creating pull requests

Using `git rebase`

Fixing rust formatting

Running pre-commit hooks

Setting up rust-analyzer

Generating documentation

Writing documentation

Creating releases

Conduct

Generating code coverage

Files

CONTRIBUTING.md

Latest commit

History

CONTRIBUTING.md

File metadata and controls

Contributing to NativeLink

Contributions

Git setup

Local development setup

Common workflows

Creating pull requests

Using git rebase

Fixing rust formatting

Running pre-commit hooks

Setting up rust-analyzer

Generating documentation

Writing documentation

Creating releases

Conduct

Generating code coverage

Using `git rebase`