Skip to content

Make a section for the tribal knowledge contained in the schema files#1216

Merged
DaedalusG merged 2 commits into
mainfrom
wg/rel/copy-tribal-from-schemas
Jun 26, 2025
Merged

Make a section for the tribal knowledge contained in the schema files#1216
DaedalusG merged 2 commits into
mainfrom
wg/rel/copy-tribal-from-schemas

Conversation

@DaedalusG
Copy link
Copy Markdown
Contributor

@DaedalusG DaedalusG commented Jun 26, 2025

closes REL-1079

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
Copy Markdown

vercel Bot commented Jun 26, 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 Jun 26, 2025 6:21am

@DaedalusG DaedalusG requested review from a team and MaedahBatool June 26, 2025 03:13
@DaedalusG DaedalusG marked this pull request as ready for review June 26, 2025 03:19
Move end marker from line 586 to line 551 to properly separate
auto-generated schema content from static educational content.

This ensures the schema sync tool only replaces the JSON schema
and preserves the Configuration Notes, Security Considerations,
and other valuable documentation.
@DaedalusG DaedalusG merged commit bc17209 into main Jun 26, 2025
5 checks passed
@DaedalusG DaedalusG deleted the wg/rel/copy-tribal-from-schemas branch June 26, 2025 16:47
@DaedalusG
Copy link
Copy Markdown
Contributor Author

@MaedahBatool Merged this one but much of this will need to be scrutinize these sections for accuracy and redundancy. This will be easier once the scehmas themselves are updated.

DaedalusG added a commit that referenced this pull request Jul 1, 2025
…ma files" (#1232)

Reverts #1216

We're not going to need these sections now with an improved schema
marshalling logic introduced here:
sourcegraph/sourcegraph#6369
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