fix!: make CommitAuthor email nullable (#799)

Author: augustoccesar

src/api/repos.rs

@@ -287,12 +287,12 @@ impl<'octo> RepoHandler<'octo> {
     ///     .branch("master")
     ///     .commiter(CommitAuthor {
     ///         name: "Octocat".to_string(),
-    ///         email: "[email protected]".to_string(),
+    ///         email: Some("[email protected]".to_string()),
     ///         date: None,
     ///     })
     ///     .author(CommitAuthor {
     ///         name: "Ferris".to_string(),
-    ///         email: "[email protected]".to_string(),
+    ///         email: Some("[email protected]".to_string()),
     ///         date: None,
     ///     })
     ///     .send()
@@ -344,12 +344,12 @@ impl<'octo> RepoHandler<'octo> {
     ///     .branch("master")
     ///     .commiter(CommitAuthor {
     ///         name: "Octocat".to_string(),
-    ///         email: "[email protected]".to_string(),
+    ///         email: Some("[email protected]".to_string()),
     ///         date: None,
     ///     })
     ///     .author(CommitAuthor {
     ///         name: "Ferris".to_string(),
-    ///         email: "[email protected]".to_string(),
+    ///         email: Some("[email protected]".to_string()),
     ///         date: None,
     ///     })
     ///     .send()
@@ -391,12 +391,12 @@ impl<'octo> RepoHandler<'octo> {
     ///     .branch("master")
     ///     .commiter(CommitAuthor {
     ///         name: "Octocat".to_string(),
-    ///         email: "[email protected]".to_string(),
+    ///         email: Some("[email protected]".to_string()),
     ///         date: None,
     ///     })
     ///     .author(CommitAuthor {
     ///         name: "Ferris".to_string(),
-    ///         email: "[email protected]".to_string(),
+    ///         email: Some("[email protected]".to_string()),
     ///         date: None,
     ///     })
     ///     .send()
@@ -775,7 +775,7 @@ impl<'octo> RepoHandler<'octo> {
     ///     .signature("signature")
     ///     .author(CommitAuthor{
     ///             name: "name".to_owned(),
-    ///             email: "email".to_owned(),
+    ///             email: Some("email".to_owned()),
     ///             date: None
     ///         })
     ///     .send()

src/api/repos/file.rs

@@ -221,12 +221,12 @@ mod tests {
             .branch("not-master")
             .commiter(CommitAuthor {
                 name: "Octocat".to_string(),
-                email: "[email protected]".to_string(),
+                email: Some("[email protected]".to_string()),
                 date: None,
             })
             .author(CommitAuthor {
                 name: "Ferris".to_string(),
-                email: "[email protected]".to_string(),
+                email: Some("[email protected]".to_string()),
                 date: None,
             });
 
@@ -260,12 +260,12 @@ mod tests {
             .branch("not-master")
             .commiter(CommitAuthor {
                 name: "Octocat".to_string(),
-                email: "[email protected]".to_string(),
+                email: Some("[email protected]".to_string()),
                 date: None,
             })
             .author(CommitAuthor {
                 name: "Ferris".to_string(),
-                email: "[email protected]".to_string(),
+                email: Some("[email protected]".to_string()),
                 date: None,
             });
 

src/models/events/payload/push.rs

@@ -45,7 +45,7 @@ mod test {
                     commit.author,
                     CommitAuthor {
                         name: "readme-bot".to_string(),
-                        email: "[email protected]".to_string(),
+                        email: Some("[email protected]".to_string()),
                         date: None,
                     }
                 );

src/models/repos.rs

@@ -158,7 +158,8 @@ pub struct GitUserTime {
 #[serde(rename_all = "snake_case")]
 pub struct CommitAuthor {
     pub name: String,
-    pub email: String,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub email: Option<String>,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub date: Option<DateTime<Utc>>,
 }