Skip to content

Conversation

@jdpleiness
Copy link
Contributor

@jdpleiness jdpleiness commented Apr 2, 2025

Unify capitalization for changelog bullets

Pull Request approval

You will need to get your PR approved by at least one member of the Sourcegraph team. For reviews of docs formatting, styles, and component usage, please tag the docs team via the #docs Slack channel.

@vercel
Copy link

vercel bot commented Apr 2, 2025

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 Apr 2, 2025 9:40pm

Copy link
Contributor Author

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

@jdpleiness jdpleiness requested review from a team and aulorbe April 2, 2025 21:23
@jdpleiness jdpleiness marked this pull request as ready for review April 2, 2025 21:23
- Clean up cmd leak `(PR #3642)`
- A few bugfixes for the experimental commit search on searcher `(PR #3639)`
- Close diff reader `(PR #3636)`
- A few small fixes for commit search on searcher experiment `(PR #3620)`
- Nightly Minor Release Pipeline `(PR #3609)`
- NA
- /latest releaseregistry endpoint returns latest by semver `(PR #3563)`
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe this is a misspellling or something? seems like releaseregistry should be 1 word

Copy link
Contributor Author

@jdpleiness jdpleiness Apr 2, 2025

Choose a reason for hiding this comment

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

It's a service name, and we generally refer to it as one word for some reason 🤷 😆

But really, it shouldn't be in there at all as it's an internal service 👀
Looks like despite being N/A in the changelog for a few of these, the author forgot to add the no-changelog label so it was still included. I cleaned up the rest of these as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

ah cool!

@jdpleiness jdpleiness force-pushed the 04-02-fix_rel_unify_changelog_caps branch from 7427025 to 6974fd9 Compare April 2, 2025 21:34
@jdpleiness jdpleiness merged commit 68bbae2 into main Apr 2, 2025
6 checks passed
@jdpleiness jdpleiness deleted the 04-02-fix_rel_unify_changelog_caps branch April 2, 2025 21:41
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.

4 participants