Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

added PHPDocs, removed unused variable #738

Merged
merged 4 commits into from
Oct 19, 2015

Conversation

daniel-mueller
Copy link
Contributor

Hi,

I added some PHPDocs and removed a unused variable.

greetings,
Daniel

@fzaninotto
Copy link
Owner

@param annotations without type and/or comments are useless IMO. Please amend your PR to add types for the phpDoc you added, or remove them.

@daniel-mueller
Copy link
Contributor Author

Thanks for feedback. I removed every @param where I couldn't safely determine the type. On the others I added a type.

@@ -15,11 +15,18 @@ public function __construct($default = null)
$this->default = $default;
}

/**
* @param string $attribute
Copy link
Owner

Choose a reason for hiding this comment

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

nah, not necessarily string. Same for __call

Copy link
Owner

Choose a reason for hiding this comment

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

Sorry, forget that.

@daniel-mueller
Copy link
Contributor Author

I added the two annotations.

fzaninotto added a commit that referenced this pull request Oct 19, 2015
added PHPDocs, removed unused variable
@fzaninotto fzaninotto merged commit f0ec03f into fzaninotto:master Oct 19, 2015
@fzaninotto
Copy link
Owner

Thanks!

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

Successfully merging this pull request may close these issues.

3 participants