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

Correct @see annotations #5

Merged
merged 1 commit into from
Aug 8, 2016
Merged

Correct @see annotations #5

merged 1 commit into from
Aug 8, 2016

Conversation

hanneskaeufler
Copy link
Contributor

It seems multiple @see annotations should not be comma-separated but listed with multiple @see-s. http://www.rubydoc.info/gems/yard/file/docs/Tags.md#see

Currently this results in a broken (and missing the others) link.

bildschirmfoto 2016-08-08 um 20 13 53

Not an expert on yard doc style at all. I went debugging the danger plugin parser and eventually settled on simply splitting the @sees lol.

It seems multiple @see annotations should not be comma-separated but listed with multiple @see-s. http://www.rubydoc.info/gems/yard/file/docs/Tags.md#see
@orta
Copy link
Owner

orta commented Aug 8, 2016

hah - think should work for me :D

@orta orta merged commit 11abeaa into orta:master Aug 8, 2016
@orta
Copy link
Owner

orta commented Aug 8, 2016

I've been wanting to do a writeup about all these gotchas once I had stabilized the danger infrastructure - danger/danger.systems#1

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.

2 participants