Merge pull request #787 from fee1-dead-contrib/add_email

Generate information about people in teams API

Author: Mark-Simulacrum

rust_team_data/src/v1.rs

@@ -176,3 +176,16 @@ pub struct Branch {
     pub ci_checks: Vec<String>,
     pub dismiss_stale_review: bool,
 }
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct Person {
+    pub name: String,
+    pub email: Option<String>,
+    pub github_id: usize,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize)]
+pub struct People {
+    /// GitHub name as key.
+    pub people: IndexMap<String, Person>,
+}

src/static_api.rs

@@ -1,5 +1,5 @@
 use crate::data::Data;
-use crate::schema::{Bot, Permissions, RepoPermission, TeamKind, ZulipGroupMember};
+use crate::schema::{Bot, Email, Permissions, RepoPermission, TeamKind, ZulipGroupMember};
 use failure::Error;
 use indexmap::IndexMap;
 use log::info;
@@ -29,6 +29,7 @@ impl<'a> Generator<'a> {
         self.generate_permissions()?;
         self.generate_rfcbot()?;
         self.generate_zulip_map()?;
+        self.generate_people()?;
         Ok(())
     }
 
@@ -306,6 +307,30 @@ impl<'a> Generator<'a> {
         Ok(())
     }
 
+    fn generate_people(&self) -> Result<(), Error> {
+        let mut people = IndexMap::new();
+
+        for person in self.data.people() {
+            people.insert(
+                person.github().into(),
+                v1::Person {
+                    name: person.name().into(),
+                    email: match person.email() {
+                        Email::Missing | Email::Disabled => None,
+                        Email::Present(s) => Some(s.into()),
+                    },
+                    github_id: person.github_id(),
+                },
+            );
+        }
+
+        people.sort_keys();
+
+        self.add("v1/people.json", &v1::People { people })?;
+
+        Ok(())
+    }
+
     fn add<T: serde::Serialize>(&self, path: &str, obj: &T) -> Result<(), Error> {
         info!("writing API object {}...", path);
         let dest = self.dest.join(path);

tests/static-api/_expected/v1/people.json

@@ -0,0 +1,39 @@
+{
+  "people": {
+    "user-0": {
+      "name": "Zeroth user",
+      "email": "user0@example.com",
+      "github_id": 0
+    },
+    "user-1": {
+      "name": "First user",
+      "email": "user1@example.com",
+      "github_id": 0
+    },
+    "user-2": {
+      "name": "Second user",
+      "email": "user2@example.com",
+      "github_id": 2
+    },
+    "user-3": {
+      "name": "Third user",
+      "email": "user3@example.com",
+      "github_id": 3
+    },
+    "user-4": {
+      "name": "Fourth user",
+      "email": "user4@example.com",
+      "github_id": 4
+    },
+    "user-5": {
+      "name": "Fifth user",
+      "email": "user5@example.com",
+      "github_id": 5
+    },
+    "user-6": {
+      "name": "Sixth user",
+      "email": "user6@example.com",
+      "github_id": 6
+    }
+  }
+}