Refactorings:

.travis.yml

@@ -9,7 +9,7 @@ matrix:
     - env: CodeQuality
       sudo: false
       language: php
-      php: 7.0
+      php: 7.1
       cache:
         directories:
           - vendor
@@ -28,7 +28,7 @@ matrix:
     - env: phpspec
       sudo: false
       language: php
-      php: 7.0
+      php: 7.1
       cache:
         directories:
           - vendor
@@ -53,7 +53,7 @@ matrix:
     - env: phpunit
       sudo: false
       language: php
-      php: 7.0
+      php: 7.1
       cache:
         directories:
           - vendor

spec/GitHubRepoSpec.php

@@ -20,15 +20,15 @@ class GitHubRepoSpec extends ObjectBehavior
     public function let(
         GitHubRepoId $id,
         GitHubRepoFullName $fullName,
-        GitHubRepoOwner $owner,
+        GitHubRepoOwner $ownerDetails,
         GitHubRepoEndpoints $endpoints,
         GitHubRepoTimestamps $timestamps,
         GitHubRepoStats $stats
     ) {
         $this->beConstructedWith(
             $id,
             $fullName,
-            $owner,
+            $ownerDetails,
             $private = false,
             $endpoints,
             $timestamps,
@@ -44,14 +44,14 @@ public function it_is_initializable()
     public function it_exposes_constructor_arguments(
         GitHubRepoId $id,
         GitHubRepoFullName $fullName,
-        GitHubRepoOwner $owner,
+        GitHubRepoOwner $ownerDetails,
         GitHubRepoEndpoints $endpoints,
         GitHubRepoTimestamps $timestamps,
         GitHubRepoStats $stats
     ) {
         $this->getId()->shouldReturn($id);
         $this->getFullName()->shouldReturn($fullName);
-        $this->getOwner()->shouldReturn($owner);
+        $this->getOwnerDetails()->shouldReturn($ownerDetails);
         $this->isPrivate()->shouldReturn(false);
         $this->isPublic()->shouldReturn(true);
         $this->getEndpoints()->shouldReturn($endpoints);

src/Commit/GitHubCommitAuthor.php

@@ -26,7 +26,7 @@ public function __construct(
         GitHubCommitAuthorName $name,
         GitHubCommitAuthorEmail $email,
         GitHubCommitDate $commitDate,
-        GitHubCommitAuthorDetails $authorDetails
+        ?GitHubCommitAuthorDetails $authorDetails
     ) {
         $this->name          = $name;
         $this->email         = $email;
@@ -49,28 +49,49 @@ public function getCommitDate(): GitHubCommitDate
         return $this->commitDate;
     }
 
-    public function getAuthorDetails(): GitHubCommitAuthorDetails
+    public function hasAuthorDetails(): bool
+    {
+        if (null === $this->authorDetails) {
+            return false;
+        }
+
+        return true;
+    }
+
+    public function getAuthorDetails(): ?GitHubCommitAuthorDetails
     {
         return $this->authorDetails;
     }
 
     public function serialize(): array
     {
+        if (true === $this->hasAuthorDetails()) {
+            $authorDetails = $this->authorDetails->serialize();
+        } else {
+            $authorDetails = null;
+        }
+
         return [
             'name'          => (string) $this->name,
             'email'         => (string) $this->email,
             'commitDate'    => (string) $this->commitDate,
-            'authorDetails' => $this->authorDetails->serialize(),
+            'authorDetails' => $authorDetails,
         ];
     }
 
     public static function deserialize(array $data): GitHubCommitAuthor
     {
+        if (null === $data['authorDetails']) {
+            $authorDetails = null;
+        } else {
+            $authorDetails = GitHubCommitAuthorDetails::deserialize($data['authorDetails']);
+        }
+
         return new self(
             new GitHubCommitAuthorName($data['name']),
             new GitHubCommitAuthorEmail($data['email']),
             new GitHubCommitDate($data['commitDate']),
-            GitHubCommitAuthorDetails::deserialize($data['authorDetails'])
+            $authorDetails
         );
     }
 }

src/Commit/GitHubCommitCommitter.php

@@ -26,7 +26,7 @@ public function __construct(
         GitHubCommitCommitterName $name,
         GitHubCommitCommitterEmail $email,
         GitHubCommitDate $commitDate,
-        GitHubCommitCommitterDetails $committerDetails
+        ?GitHubCommitCommitterDetails $committerDetails
     ) {
         $this->name             = $name;
         $this->email            = $email;
@@ -49,28 +49,49 @@ public function getCommitDate(): GitHubCommitDate
         return $this->commitDate;
     }
 
-    public function getCommitterDetails(): GitHubCommitCommitterDetails
+    public function hasCommitterDetails(): bool
+    {
+        if (null === $this->committerDetails) {
+            return false;
+        }
+
+        return true;
+    }
+
+    public function getCommitterDetails(): ?GitHubCommitCommitterDetails
     {
         return $this->committerDetails;
     }
 
     public function serialize(): array
     {
+        if (true === $this->hasCommitterDetails()) {
+            $committerDetails = $this->committerDetails->serialize();
+        } else {
+            $committerDetails = null;
+        }
+
         return [
             'name'             => (string) $this->name,
             'email'            => (string) $this->email,
             'commitDate'       => (string) $this->commitDate,
-            'committerDetails' => $this->committerDetails->serialize(),
+            'committerDetails' => $committerDetails,
         ];
     }
 
     public static function deserialize(array $data): GitHubCommitCommitter
     {
+        if (null === $data['committerDetails']) {
+            $committerDetails = null;
+        } else {
+            $committerDetails = GitHubCommitCommitterDetails::deserialize($data['committerDetails']);
+        }
+
         return new self(
             new GitHubCommitCommitterName($data['name']),
             new GitHubCommitCommitterEmail($data['email']),
             new GitHubCommitDate($data['commitDate']),
-            GitHubCommitCommitterDetails::deserialize($data['committerDetails'])
+            $committerDetails
         );
     }
 }

src/GitHubRepo.php

@@ -24,7 +24,7 @@ class GitHubRepo
     /** @var GitHubRepoFullName */
     private $fullName;
     /** @var GitHubRepoOwner */
-    private $owner;
+    private $ownerDetails;
     /** @var bool */
     private $private;
     /** @var GitHubRepoEndpoints */
@@ -37,19 +37,19 @@ class GitHubRepo
     public function __construct(
         GitHubRepoId $id,
         GitHubRepoFullName $fullName,
-        GitHubRepoOwner $owner,
+        ?GitHubRepoOwner $ownerDetails,
         bool $private,
         GitHubRepoEndpoints $endpoints,
         GitHubRepoTimestamps $timestamps,
         GitHubRepoStats $stats
     ) {
-        $this->id         = $id;
-        $this->fullName   = $fullName;
-        $this->owner      = $owner;
-        $this->private    = $private;
-        $this->endpoints  = $endpoints;
-        $this->timestamps = $timestamps;
-        $this->stats      = $stats;
+        $this->id           = $id;
+        $this->fullName     = $fullName;
+        $this->ownerDetails = $ownerDetails;
+        $this->private      = $private;
+        $this->endpoints    = $endpoints;
+        $this->timestamps   = $timestamps;
+        $this->stats        = $stats;
     }
 
     public function getId(): GitHubRepoId
@@ -72,9 +72,18 @@ public function getRepoName(): GitHubRepoName
         return $this->fullName->getRepoName();
     }
 
-    public function getOwner(): GitHubRepoOwner
+    public function hasOwnerDetails(): bool
     {
-        return $this->owner;
+        if (null === $this->ownerDetails) {
+            return false;
+        }
+
+        return true;
+    }
+
+    public function getOwnerDetails(): ?GitHubRepoOwner
+    {
+        return $this->ownerDetails;
     }
 
     public function isPrivate(): bool
@@ -104,23 +113,35 @@ public function getStats(): GitHubRepoStats
 
     public function serialize(): array
     {
+        if (true === $this->hasOwnerDetails()) {
+            $ownerDetails = $this->ownerDetails->serialize();
+        } else {
+            $ownerDetails = null;
+        }
+
         return [
-            'id'         => $this->id->getValue(),
-            'fullName'   => (string) $this->fullName,
-            'owner'      => $this->owner->serialize(),
-            'private'    => $this->private,
-            'endpoints'  => $this->endpoints->serialize(),
-            'timestamps' => $this->timestamps->serialize(),
-            'stats'      => $this->stats->serialize(),
+            'id'           => $this->id->getValue(),
+            'fullName'     => (string) $this->fullName,
+            'ownerDetails' => $ownerDetails,
+            'private'      => $this->private,
+            'endpoints'    => $this->endpoints->serialize(),
+            'timestamps'   => $this->timestamps->serialize(),
+            'stats'        => $this->stats->serialize(),
         ];
     }
 
     public static function deserialize(array $data): GitHubRepo
     {
+        if (null === $data['ownerDetails']) {
+            $ownerDetails = null;
+        } else {
+            $ownerDetails = GitHubRepoOwner::deserialize($data['ownerDetails']);
+        }
+
         return new self(
             new GitHubRepoId($data['id']),
             GitHubRepoFullName::createFromString($data['fullName']),
-            GitHubRepoOwner::deserialize($data['owner']),
+            $ownerDetails,
             $data['private'],
             GitHubRepoEndpoints::deserialize($data['endpoints']),
             GitHubRepoTimestamps::deserialize($data['timestamps']),

tests/Commit/GitHubCommitAuthorTest.php

@@ -30,7 +30,7 @@ public function testCreating(
         GitHubCommitAuthorName $name,
         GitHubCommitAuthorEmail $email,
         GitHubCommitDate $commitDate,
-        GitHubCommitAuthorDetails $authorDetails
+        ?GitHubCommitAuthorDetails $authorDetails
     ) {
         $sut = new GitHubCommitAuthor($name, $email, $commitDate, $authorDetails);
 
@@ -45,7 +45,7 @@ public function testSerializationAndDeserialization(
         GitHubCommitAuthorName $name,
         GitHubCommitAuthorEmail $email,
         GitHubCommitDate $commitDate,
-        GitHubCommitAuthorDetails $authorDetails
+        ?GitHubCommitAuthorDetails $authorDetails
     ) {
         $sut = new GitHubCommitAuthor($name, $email, $commitDate, $authorDetails);
 
@@ -72,6 +72,12 @@ public function provideArguments(): array
                     false
                 ),
             ],
+            [
+                new GitHubCommitAuthorName('name'),
+                new GitHubCommitAuthorEmail('nobody@example.com'),
+                new GitHubCommitDate('2017-02-03 11:22:33'),
+                null,
+            ],
         ];
     }
 }

tests/Commit/GitHubCommitCommitterTest.php

@@ -30,7 +30,7 @@ public function testCreating(
         GitHubCommitCommitterName $name,
         GitHubCommitCommitterEmail $email,
         GitHubCommitDate $commitDate,
-        GitHubCommitCommitterDetails $committerDetails
+        ?GitHubCommitCommitterDetails $committerDetails
     ) {
         $sut = new GitHubCommitCommitter($name, $email, $commitDate, $committerDetails);
 
@@ -45,7 +45,7 @@ public function testSerializationAndDeserialization(
         GitHubCommitCommitterName $name,
         GitHubCommitCommitterEmail $email,
         GitHubCommitDate $commitDate,
-        GitHubCommitCommitterDetails $committerDetails
+        ?GitHubCommitCommitterDetails $committerDetails
     ) {
         $sut = new GitHubCommitCommitter($name, $email, $commitDate, $committerDetails);
 
@@ -72,6 +72,12 @@ public function provideArguments(): array
                     false
                 ),
             ],
+            [
+                new GitHubCommitCommitterName('name'),
+                new GitHubCommitCommitterEmail('nobody@example.com'),
+                new GitHubCommitDate('2017-02-03 11:22:33'),
+                null,
+            ],
         ];
     }
 }