jscs-dev / jscs-jsdoc

JsDoc validation rules for jscs
MIT License
99 stars 35 forks source link

requireParamTypes and argument with properties #132

Closed markelog closed 9 years ago

markelog commented 9 years ago

From https://code.google.com/p/jsdoc-toolkit/wiki/TagParam

/**
  * @param userInfo Information about the user.
  * @param userInfo.name The name of the user.
  * @param userInfo.email The email of the user.
  */

But jsDoc will warn for such definitions

qfox commented 9 years ago

@markelog I don't see it's broken ;-( Need more info. See #133

markelog commented 9 years ago

Did you run checkParamNames on code:

 /**
  * @param userInfo Information about the user.
  * @param userInfo.name The name of the user.
  * @param userInfo.email The email of the user.
  */
 function logIn(userInfo) {
        doLogIn(userInfo.name, userInfo.email);
 }

?

qfox commented 9 years ago

I've updated test, still no error ;-(

https://github.com/jscs-dev/jscs-jsdoc/pull/133/files#diff-4170383ccbd6bc30dfa2c057c8393ce9R484

markelog commented 9 years ago

Sorry, it seems requireParamTypes is the one offending rule

qfox commented 9 years ago

Hm, does requireParamTypes not works as expected? What you suggest to do? ;-)

markelog commented 9 years ago

No, i'm just stupid!

qfox commented 9 years ago

;-( okay.