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

Fix display of section comments #373

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jmalcic
Copy link

@jmalcic jmalcic commented Dec 27, 2024

This PR addresses an issue resulting from RDoc having changed where comments are stored in sections (since ruby/rdoc@4b50423 as an array at RDoc::Context::Section#comments) without removing the old reader. This can make it seem like #comment is still available as with other RDoc objects, when in reality for sections it now never has a value. This then means #description doesn't work for sections, because it calls the old reader. I couldn't find any tests in the RDoc repo suggesting how #comments is supposed to be formatted, so this was just a guess

@jmalcic jmalcic changed the title Display section comments Fix display of section comments Dec 27, 2024
Copy link

netlify bot commented Dec 27, 2024

Deploy Preview for sdoc failed.

Name Link
🔨 Latest commit 43e2184
🔍 Latest deploy log https://app.netlify.com/sites/sdoc/deploys/676ecfa59e55540008821c14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant