Skip to content

Move method doc comments to inside {% begin %} [fixup #16034]#16047

Merged
straight-shoota merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:restore-method-docs
Aug 4, 2025
Merged

Move method doc comments to inside {% begin %} [fixup #16034]#16047
straight-shoota merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:restore-method-docs

Conversation

@Blacksmoke16
Copy link
Member

Noticed that after #16034, none of the methods that had the deprecated parameter had docs anymore. I think the gist of it is {% begin %} leaves an empty line so the docs aren't actually applied to the method.

@Blacksmoke16 Blacksmoke16 added the kind:bug A bug in the code. Does not apply to documentation, specs, etc. label Aug 2, 2025
@straight-shoota straight-shoota changed the title Move method doc comments to inside {% begin %} Move method doc comments to inside {% begin %} [fixup #16034] Aug 2, 2025
@straight-shoota straight-shoota added this to the 1.18.0 milestone Aug 2, 2025
@straight-shoota straight-shoota merged commit 4a5d94d into crystal-lang:master Aug 4, 2025
40 checks passed
@straight-shoota straight-shoota added kind:docs topic:stdlib:files and removed kind:bug A bug in the code. Does not apply to documentation, specs, etc. labels Aug 4, 2025
@Blacksmoke16 Blacksmoke16 deleted the restore-method-docs branch December 5, 2025 23:34
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.

2 participants