only check part of changed github profiles (#1035)

wechaty · Jul 25, 2021 · 208717f · 208717f
1 parent e03267e
commit 208717f
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 6 deletions.
diff --git a/jekyll/_contributors/atorber.md b/jekyll/_contributors/atorber.md
@@ -2,6 +2,6 @@
 name: atorber
 site: <https://github.com/atorber>
 bio: 一个酷爱编码的产品经理
-avatar: /assets/contributors/atorber/avatar.png
+avatar: /assets/contributors/atorber/avatar.webp
 email: [email protected]
----
+---
diff --git a/src/jekyll/get-frontmatter-avatar-list.spec.ts b/src/jekyll/get-frontmatter-avatar-list.spec.ts
@@ -14,7 +14,7 @@ test('getFrontmatterAvatarList', async t => {
     'huan.md',
   )
   const EXPECTED_AVATAR_NUM = 1
-  const EXPECTED_AVATAR_PATH = '/assets/contributors/huan/avatar.jpg'
+  const EXPECTED_AVATAR_PATH = '/assets/contributors/huan/avatar.webp'
 
   const imageList = getFrontmatterAvatarList(FILE)
 

diff --git a/tests/jekyll/contributor-profile.spec.ts b/tests/jekyll/contributor-profile.spec.ts
@@ -8,7 +8,10 @@ import util         from 'util'
 import fs     from 'fs'
 import globCB from 'glob'
 import { loadFront } from 'yaml-front-matter'
-import { chunk } from 'lodash'
+import {
+  chunk,
+  shuffle,
+}               from 'lodash'
 
 import {
   JEKYLL_FOLDER,
@@ -104,14 +107,25 @@ test('developer profile name must be GitHub username', async t => {
     since: '1 week ago',
   })
 
+  const needToBeChecked = (file: string) => changedFileList.includes(file)
+
   // console.info('changedFileList', changedFileList)
   // console.info('allContributorsFileList', allContributorsFileList)
 
-  const urlList = allContributorsFileList
-    .filter(file => changedFileList.includes(file))
+  let urlList = allContributorsFileList
+    .filter(needToBeChecked)
     .map(contributorFilenameToUsername)
     .map(name => `https://github.com/${name}`)
 
+  /**
+   * Huan(202107): Only check part of them because of the limitation of GitHub API
+   */
+  const MAX_NUM = 50
+  if (urlList.length > MAX_NUM) {
+    urlList = shuffle(urlList)
+      .slice(0, MAX_NUM)
+  }
+
   // console.info(urlList)
 
   const nameListChunk = chunk(urlList, 10)