Skip to content

Conversation

@michaellzc
Copy link
Member

Pull Request approval

Although pull request approval is not enforced for this repository in order to reduce friction, merging without a review will generate a ticket for the docs team to review your changes. So if possible, have your pull request approved before merging.

@vercel
Copy link

vercel bot commented Sep 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 25, 2024 5:33pm

Copy link
Member Author

michaellzc commented Sep 24, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @michaellzc and the rest of your teammates on Graphite Graphite

Copy link
Contributor

@loujar loujar left a comment

Choose a reason for hiding this comment

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

I think we should also update the Configuration sections in each of the individual code host's doc pages

@MaedahBatool MaedahBatool changed the base branch from main to sg-next-oct2024 September 25, 2024 16:46
@MaedahBatool
Copy link
Contributor

@michaellzc thanks for the PR. I have changed the PR base to our docs working branch. There's some feedback from @loujar. Let me know when it's done. Appreciate your help here. 🙏

@michaellzc
Copy link
Member Author

I think we should also update the Configuration sections in each of the individual code host's doc pages

are you talking about the embedded jsonschema? I believe some automation is responsible to update them during release time.

@michaellzc michaellzc force-pushed the 09-24-update_ssh_repo_clone_docs branch from ac08d51 to 5185d87 Compare September 25, 2024 17:31
@michaellzc michaellzc merged commit 78045d4 into sg-next-oct2024 Sep 25, 2024
6 checks passed
@michaellzc michaellzc deleted the 09-24-update_ssh_repo_clone_docs branch September 25, 2024 17:36
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.

5 participants