Do little code/phpdocs fixes

Author: l3l0

lib/Github/Api/CurrentUser/Emails.php

@@ -13,7 +13,7 @@ class Emails extends AbstractApi
 {
     /**
      * List emails for the authenticated user
-     * @link http://developer.github.com/v3/repos/emails/
+     * @link http://developer.github.com/v3/users/emails/
      *
      * @return array
      */
@@ -24,7 +24,7 @@ public function all()
 
     /**
      * Adds one or more email for the authenticated user
-     * @link http://developer.github.com/v3/repos/emails/
+     * @link http://developer.github.com/v3/users/emails/
      *
      * @param  string|array $emails
      * @return array
@@ -44,7 +44,7 @@ public function add($emails)
 
     /**
      * Removes one or more email for the authenticated user
-     * @link http://developer.github.com/v3/repos/emails/
+     * @link http://developer.github.com/v3/users/emails/
      *
      * @param  string|array $emails
      * @return array
@@ -59,6 +59,6 @@ public function remove($emails)
             throw new InvalidArgumentException();
         }
 
-        return $this->delete('user/emails/', $emails);
+        return $this->delete('user/emails', $emails);
     }
 }

lib/Github/Api/Issue/Events.php

@@ -12,7 +12,7 @@ class Events extends AbstractApi
 {
     public function all($username, $repository, $issue = null, $page = 1)
     {
-        if (null === $issue) {
+        if (null !== $issue) {
             return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/issues/'.urlencode($issue).'/events', array(
                 'page' => $page
             ));

lib/Github/Api/Issue/Milestones.php

@@ -13,13 +13,13 @@ class Milestones extends AbstractApi
 {
     public function all($username, $repository, array $params = array())
     {
-        if (!in_array($params['state'], array('open', 'closed'))) {
+        if (isset($params['state']) && !in_array($params['state'], array('open', 'closed'))) {
             $params['state'] = 'open';
         }
-        if (!in_array($params['sort'], array('due_date', 'completeness'))) {
+        if (isset($params['sort']) && !in_array($params['sort'], array('due_date', 'completeness'))) {
             $params['sort'] = 'due_date';
         }
-        if (!in_array($params['direction'], array('asc', 'desc'))) {
+        if (isset($params['direction']) && !in_array($params['direction'], array('asc', 'desc'))) {
             $params['direction'] = 'desc';
         }
 

lib/Github/Api/Repository/Contents.php

@@ -13,9 +13,7 @@ class Contents extends AbstractApi
 {
     public function readme($username, $repository, $reference = null)
     {
-        return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/contents/readme', array(
-            'ref' => $reference
-        ));
+        return $this->show($username, $repository, 'readme', $reference);
     }
 
     public function show($username, $repository, $path, $reference = null)

lib/Github/HttpClient/HttpClient.php

@@ -85,7 +85,7 @@ public function authenticate()
         $this->browser->addListener(
             new AuthListener(
                 $this->options['auth_method'],
-                array($this->options['login'], $this->options['password'], $this->options['token'])
+                array('login' => $this->options['login'], 'password' => $this->options['password'], 'token' => $this->options['token'])
             )
         );
     }
@@ -96,6 +96,8 @@ public function authenticate()
     public function setHeaders(array $headers)
     {
         $this->headers = $headers;
+
+        return $this;
     }
 
     /**

lib/Github/HttpClient/HttpClientInterface.php

@@ -83,6 +83,7 @@ public function setOption($name, $value);
      * Set HTTP headers
      *
      * @param array $headers
+     * @return HttpClientInterface The current object instance
      */
     public function setHeaders(array $headers);
 }