Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update development docs to include bazel update instructions #1118

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

willdollman
Copy link
Contributor

@willdollman willdollman commented Oct 16, 2024

The current documentation contains an outdated go generate step (now requires sg generate bazel), and doesn't include instructions for updating Bazel's src-cli reference.

This PR updates the development Releasing docs to include the new required step.

For an example of a PR that running these new steps results in, see https://github.com/sourcegraph/sourcegraph/pull/1112

Test plan

  • Test instructions locally

@willdollman willdollman self-assigned this Oct 16, 2024
@willdollman willdollman marked this pull request as ready for review October 16, 2024 15:01
@willdollman willdollman requested a review from a team as a code owner October 16, 2024 15:01
Copy link
Member

@camdencheek camdencheek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants