Skip to content

Add generate-docc-reference plugin#694

Merged
rauhul merged 4 commits into
mainfrom
generate-docc-reference
Feb 5, 2025
Merged

Add generate-docc-reference plugin#694
rauhul merged 4 commits into
mainfrom
generate-docc-reference

Conversation

@rauhul

@rauhul rauhul commented Feb 5, 2025

Copy link
Copy Markdown
Collaborator

Upstreams swiftly's generate-docs-plugin with minimal changes. Adds
tests against argument-parser's built in example tools.

The initial version of this tool is extremely minimal and should be
extended to output much more information contained in tool info, like
generate-manual does.

@rauhul

rauhul commented Feb 5, 2025

Copy link
Copy Markdown
Collaborator Author

@swift-ci test

@rauhul rauhul added the enhancement New feature or request label Feb 5, 2025
@rauhul

rauhul commented Feb 5, 2025

Copy link
Copy Markdown
Collaborator Author

@swift-ci test

return ""
}

// TODO: default values, short, etc.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thought: It'll be interesting to see how these should be handled in the general case.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Definitely, I've pinged the documentation workgroup on slack to see if they general ideas.

return ""
}

// TODO: default values, values, short, etc.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thought: and same here

Upstreams swiftly's generate-docs-plugin with minimal changes. Adds
tests against argument-parser's built in example tools.

The initial version of this tool is extremely minimal and should be
extended to output much more information contained in tool info, like
generate-manual does.
@rauhul rauhul force-pushed the generate-docc-reference branch from f9e9050 to 7f294c0 Compare February 5, 2025 17:10

@main
struct GenerateManual: ParsableCommand {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

question: Maybe this change doesn't need to be here?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yeah but it annoyed me and I didn't feel like an independent PR to fix just that line 🙈

I want this stuff to handled by a formatting workflow but I need the infra owners to turn on GitHub actions. Which I keep asking for.

@natecook1000 natecook1000 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The tiniest possible nit, otherwise LGTM 👍🏻

Comment thread Package@swift-5.8.swift Outdated
rauhul and others added 2 commits February 5, 2025 09:40
@rauhul rauhul enabled auto-merge (squash) February 5, 2025 17:42
@rauhul

rauhul commented Feb 5, 2025

Copy link
Copy Markdown
Collaborator Author

@swift-ci test

@rauhul rauhul merged commit 9289e31 into main Feb 5, 2025
@rauhul rauhul deleted the generate-docc-reference branch February 5, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants