Skip to content

Conversation

eseliger
Copy link
Member

I found it a bit chaotic to move between several pages for a feature as simple as code monitoring, so I reduced some redundant content and consolidated it to a length where I feel ok about having it be a single page only. I think there's more potential for simplification here, but this feels like a good improvement regardless.

Test plan: Review.

I found it a bit chaotic to move between several pages for a feature as simple as code monitoring, so I reduced some redundant content and consolidated it to a length where I feel ok about having it be a single page only. I think there's more potential for simplification here, but this feels like a good improvement regardless.

Test plan: Review.
Copy link

vercel bot commented Jul 28, 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 Jul 28, 2025 6:22pm

@eseliger eseliger marked this pull request as ready for review July 28, 2025 18:35
@eseliger
Copy link
Member Author

joe please stamp

@eseliger eseliger merged commit dc9be56 into main Jul 28, 2025
7 checks passed
@eseliger eseliger deleted the es/cm-merge branch July 28, 2025 18:58
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