Skip to content

Commit

Permalink
Update ManagesFrequencies.php
Browse files Browse the repository at this point in the history
  • Loading branch information
taylorotwell authored Jul 15, 2019
1 parent a2016c3 commit 084546d
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/Illuminate/Console/Scheduling/ManagesFrequencies.php
Original file line number Diff line number Diff line change
Expand Up @@ -122,10 +122,9 @@ public function hourly()
}

/**
* Schedule the event to run hourly at a given offset in the hour. The
* offset can be either an integer or an array of integers.
* Schedule the event to run hourly at a given offset in the hour.
*
* @param int|array $offset
* @param array|int $offset

This comment has been minimized.

Copy link
@jrebs

jrebs Jul 15, 2019

Contributor

Hi Taylor, thanks for merging my PR. Just so I have this correct in the future, is the intention that all of the param types in the docblock should be listed alphabetically? Even if some are application classes? For instance:

@param int|MySpecialClass

This comment has been minimized.

Copy link
@driesvints

driesvints Jul 16, 2019

Member

@jrebs no but it's more common to first list the array type in the framework I guess.

* @return $this
*/
public function hourlyAt($offset)
Expand Down

0 comments on commit 084546d

Please sign in to comment.