Skip to content

Commit

Permalink
Update development docs to include bazel update instructions
Browse files Browse the repository at this point in the history
  • Loading branch information
willdollman committed Oct 16, 2024
1 parent a9118d5 commit 4e174cc
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions DEVELOPMENT.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,14 @@ We adhere to the [general Sourcegraph principles for testing](https://docs.sourc
3. The [releases section of the repo sidebar](https://github.com/sourcegraph/src-cli) shows the correct version.
5. Make the necessary updates to the main Sourcegraph repo:
1. Update the `MinimumVersion` constant in the [src-cli package](https://github.com/sourcegraph/sourcegraph/tree/main/internal/src-cli/consts.go).
2. Update the reference documentation by running `go generate ./doc/cli/references`.
3. Commit the changes, and open a PR.
2. Update the `SRC_CLI_VERSION` in [tool_deps.bzl](https://sourcegraph.sourcegraph.com/github.com/sourcegraph/sourcegraph@f88ee515885e1761f002f8adcc73560bcb2573ee/-/blob/dev/tool_deps.bzl?L6)
3. Update the sha256 hashes for the [three `src-cli-*` targets](https://sourcegraph.sourcegraph.com/github.com/sourcegraph/sourcegraph@f88ee515885e1761f002f8adcc73560bcb2573ee/-/blob/dev/tool_deps.bzl?L113-133) in this file. The easiest way to do this is to run the following command after updating `SRC_CLI_VERSION`, and extract the correct checksums for each target from the error message:
```shell
$ bazel build @src-cli-linux-amd64//:src-cli-linux-amd64 @src-cli-darwin-amd64//:src-cli-darwin-amd64 @src-cli-darwin-arm64//:src-cli-darwin-arm64`

ERROR: java.io.IOException: Error downloading [...] Checksum was [x]
```
4. Commit the changes, and open a PR.
6. Once the version bump PR is merged and the commit is live on dotcom, check that the [curl commands in the README](README.md#installation) also fetch the new latest version.
### Patch releases
Expand Down

0 comments on commit 4e174cc

Please sign in to comment.