Skip to content

[pull] master from ruby:master#139

Merged
pull[bot] merged 1 commit intosysfce2:masterfrom
ruby:master
Feb 8, 2026
Merged

[pull] master from ruby:master#139
pull[bot] merged 1 commit intosysfce2:masterfrom
ruby:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Feb 8, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

I'm currently trying to group methods in the prism docs by topic like
"location related", " flag related", etc. so that methods not related to
these conceps show up more prominently (not mixed in with potentially
many other methods). But I noticed that sections do not currently show
their comment.

There were a few things wrong that made this not work:
* Formatting expects `@comment` but there is only `@comments`
* The parse method used for marshaling overwrote the markdown parse
method
* `@store` was not present, which the formatter requires

Comments can be added/removed, so there still needs to be the array to
make it possible to properly modify the section.

Fixes #345, Closes #936

On https://docs.ruby-lang.org/en/master/Prism/Node.html#node-interface
there is actually already a comment but it gets swallowed. With this
change, it gets properly rendered:

<img width="772" height="143" alt="image"
src="https://github.com/user-attachments/assets/70885890-8ebc-48a8-9fde-f2fc8526cee1"
/>
@pull pull Bot locked and limited conversation to collaborators Feb 8, 2026
@pull pull Bot added the ⤵️ pull label Feb 8, 2026
@pull pull Bot merged commit ba21821 into sysfce2:master Feb 8, 2026
4 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant