Skip to content
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

Expose doc comments on ASTNode when generating docs #14050

Merged
merged 4 commits into from
Dec 8, 2023

Conversation

Blacksmoke16
Copy link
Member

@Blacksmoke16 Blacksmoke16 commented Dec 5, 2023

No description provided.

@straight-shoota
Copy link
Member

Does this depend on #14048 in any other way than that it is mentioned in the documentation as a typical use case?

@Blacksmoke16
Copy link
Member Author

No, just referenced in the docs. At least wanted to call that out as if something came up where the other was delayed, this one should have that changed to avoid confusing docs if they dont make it into the same release.

@straight-shoota
Copy link
Member

Then let's remove that dependency. It's easier to focus on each individual change independently.
We keep track of the entire integration in #14039

Can be re-added within a follow up PR once both this and that PR are merged
@Blacksmoke16
Copy link
Member Author

@straight-shoota Done. I removed the references from the macro docs. Can be re-added within a follow up PR once both this and #14048 are merged.

@straight-shoota straight-shoota linked an issue Dec 6, 2023 that may be closed by this pull request
@straight-shoota straight-shoota added this to the 1.11.0 milestone Dec 6, 2023
@straight-shoota straight-shoota changed the title Expose the documentation comments on an ASTNode when generating docs Expose doc comments on ASTNode when generating docs Dec 8, 2023
@straight-shoota straight-shoota merged commit 0920944 into crystal-lang:master Dec 8, 2023
54 of 55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add TypeNode#docs method
3 participants