Skip to content

Add ngdoc annotations to mod2.js#1

Open
btesser wants to merge 1 commit intopetebacondarwin:masterfrom
btesser:patch-1
Open

Add ngdoc annotations to mod2.js#1
btesser wants to merge 1 commit intopetebacondarwin:masterfrom
btesser:patch-1

Conversation

@btesser
Copy link

@btesser btesser commented Oct 21, 2014

Using new alias syntax... maybe I should hold off and use name though

Using new alias syntax... maybe I should hold off and use name though
@petebacondarwin
Copy link
Owner

The aim os to minimise the number of tags we need to write

@btesser
Copy link
Author

btesser commented Oct 21, 2014

Awesome.... I like that!

@btesser
Copy link
Author

btesser commented Oct 21, 2014

Does it work when this syntax is followed:

/**
 * Controller which does something with scope
 */
function MyController($scope) {
  $scope.peanuts  = 'food';
}

/**
 * @param {Array.<String>} toppings Pizza toppings
 * @returns {Boolean}
 */
MyController.prototype.eatPizza = function (toppings) {
  return toppings && toppings.length && toppings.indexOf('pepperoni') > -1;
};

angular.module('my.module').controller('MyController', MyController);

Most of my code follows a structure like that. PS After reading that code snippet it's probably clear I'm hungry...

@petebacondarwin
Copy link
Owner

Not yet. Je pense. But I will work on it
On 21 Oct 2014 13:09, "Ben Tesser" notifications@github.com wrote:

Does it work when this syntax is followed:

/** * Controller which does something with scope /function MyController($scope) {
$scope.peanuts = 'food';}
/
* * @param {Array.} toppings Pizza toppings * @returns {Boolean} */MyController.prototype.eatPizza = function (toppings) {
return toppings && toppings.length && toppings.indexOf('pepperoni') > -1;};
angular.module('my.module').controller('MyController', MyController);

Most of my code follows a structure like that. PS After reading that code
snippet it's probably clear I'm hungry...


Reply to this email directly or view it on GitHub
#1 (comment)
.

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