-
Notifications
You must be signed in to change notification settings - Fork 167
Experimental markdown output #1303
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
Merged
d-ronnqvist
merged 67 commits into
swiftlang:main
from
jrturton:159600318/experimental-markdown-ouput
Dec 19, 2025
Merged
Changes from all commits
Commits
Show all changes
67 commits
Select commit
Hold shift + click to select a range
cf7ebc6
Add experimental markdown output flag and pass it through to the conv…
4b1b94a
Initial export of Markdown from Article
55e7836
Initial processing of a type-level symbol
53a6196
Adds symbol declarations and article reference links
3513a73
Output tutorials to markdown
81d2d5a
Be smarter about removing indentation from within block directives
f3fa5ab
Baseline for adding new tests for markdown output
5013a09
Basic test infrastructure for markdown output
6798162
Adds symbol link tests to markdown output
132cd6c
Tutoorial code rendering markdown tests
1753cca
Adding metadata to markdown output
301d7da
Include package source for markdown output test catalog
9607e3b
Output metadata updates
5244f0f
Adds default availability for modules to markdown export
a6a740e
Move availability out of symbol and in to general metadata for markdo…
d0dbf44
Refactor markdown output so the final node type is standalone
595831a
Add generated markdown flag to render node metadata
104f9eb
Only include unavailable in markdown header if it is true
a2aa8f1
Initial setup of manifest output, no references
b5ed559
Output of manifest / relationships
74b6023
Manifest output format updates
198d8e8
Remove member symbol relationship
53ba222
More compact availability, deal with metadata availability for symbols
af890be
Merge branch 'main' into 159600318/experimental-markdown-ouput
jrturton 4784380
Update tests so all inputs are locally defined
abc9985
Remove print statements from unused visitors
edc74da
Merge branch 'main' into 159600318/experimental-markdown-ouput
eb77d39
Added snippet handling
0e867d7
Remove or _spi-hide new public API
f07f683
Remove or _spi-hide new public API (part 2)
6ccfe8f
Prevent recursion when the abstract of a link contains a link and we …
4fcd9fe
Test and temporary fix for colspan issue https://github.com/swiftlang…
ba716ed
Remove separate SwiftDocCMarkdownOutput target
5c835b0
Bump swift-markdown, remove temporary bug fix
85ea19b
Merge branch 'main' into 159600318/experimental-markdown-ouput
43efc75
Updates to handle removed bundle parameter
8c15b03
Extract common writer code into a helper function
4bd8267
Clarify use of article.md in test catalog
4a8b196
Deal with internal links with anchor tags
fc6cf5c
Deal with unresolved symbol links
778d33a
Updates to improve public API / clarity of MarkdownOutputNode
e23115a
Improve public API of manifest, sort contents before writing to preve…
3659560
Don't use unstructured `Data` for WritableMarkdownOutputNode
ff960cd
Reduce public SPI surface around markdown writing
53ae33b
Make relationship subtype a specific type rather than a string
c0d356c
Make it clear that path component is a fallback title, harden linked …
a85e59f
Add todos and missing tests
21c1165
Merge branch 'main' into 159600318/experimental-markdown-ouput
jrturton 293246a
Move markdown output types from public SPI to package
737be98
Merge branch 'main' into 159600318/experimental-markdown-ouput
33fa249
don't include scheme or host in documentation links
9481ccf
Reformat multi-condition if statements
0df68dd
remove whitespace-only change
172674c
Remove unused declaration
8a91913
Remove redundant return types
f9331f5
Remove MarkdownOutputNodeTranslator
c9f9131
Add TODO for alternate declarations
5a4cedf
Add TODO for structural review
ae69449
Add stacks todo, set writable type back to package
0a4c251
uri -> identifier
86e40fa
Use RelationshipsGroup.Kind instead of creating a mirror type
fcb3734
Merge branch 'main' into 159600318/experimental-markdown-ouput
jrturton 78361eb
Lower access level of manifest, remove children method
71916e7
Restrict package level declarations to those actually used at package…
562fcc4
Merging main
53def00
Merge branch 'main' into 159600318/experimental-markdown-ouput
8cfa398
Merge branch 'main' into 159600318/experimental-markdown-ouput
d-ronnqvist File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: what's the reason for including the title in the manifest file?
Have I misunderstood that archives would only have a single manifest file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One archive will only have a single manifest file, but if you are processing the output of multiple archives this allows you to distinguish them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't the caller already know what archive the manifest belongs to based on what archive the manifest file is inside?