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

Adds @implements and @interface tags #357

Merged
merged 1 commit into from
Jun 24, 2015
Merged

Adds @implements and @interface tags #357

merged 1 commit into from
Jun 24, 2015

Conversation

mzummo
Copy link
Contributor

@mzummo mzummo commented Jun 19, 2015

Adds tags to suppress warnings on doc build. Can be parsed out as well from json docs

Signed-off-by: Mathew Zummo mzummo@gmail.com

@yahoocla
Copy link

CLA is valid!

@okuryu
Copy link
Member

okuryu commented Jun 21, 2015

I'm not able to verify that this fixes the warning. Did you tested this?

@mzummo
Copy link
Contributor Author

mzummo commented Jun 22, 2015

yes i tested it, a complete implementation of @interface @implements would be nice but this will work for now, should I open a issue for that?

@mzummo
Copy link
Contributor Author

mzummo commented Jun 22, 2015

@okuryu i see the prob, i accidentally put an @ in the string in docparser should of been just "implements" vs "@implements"

'writeonce' // YUI attribute config
'writeonce', // YUI attribute config
'implements', // Implements Interface
'interface' // Is Interface
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you make alphanumerics sort?

Signed-off-by: Mathew Zummo <mzummo@gmail.com>
@mzummo
Copy link
Contributor Author

mzummo commented Jun 23, 2015

@okuryu should be good to go

@okuryu okuryu merged commit 329207d into yui:master Jun 24, 2015
@okuryu
Copy link
Member

okuryu commented Jun 24, 2015

Merged. Thanks!

@okuryu okuryu modified the milestone: v0.8.0 Jul 5, 2015
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.

3 participants