Merge pull request #475 from adarsh-jha-dev/390-Automate-Badge-Distri…

…bution-Based-on-Merged-PR-Activity-bug-fix typo in yml file
Chimoney · Nov 28, 2024 · d9ffd4d · d9ffd4d
2 parents 8db3f55 + 5d1b69f
commit d9ffd4d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/.github/scripts/badge-automation.js b/.github/scripts/badge-automation.js
@@ -77,9 +77,9 @@ async function sendEmail(email, message, badgeInfo) {
 async function getUserEmail(username) {
   try {
     const { data: user } = await octokit.users.getByUsername({ username });
-    return user.email;
+    return user.email || null;
   } catch (error) {
-    console.error(`Error fetching email for ${username}:`, error);
+    console.error(`Error fetching email for ${username}:`, error.message);
     return null;
   }
 }

diff --git a/.github/workflows/badge-automation.yml b/.github/workflows/badge-automation.yml
@@ -18,12 +18,11 @@ jobs:
           node-version: "16.x"
 
       - name: Install dependencies
-        run: npm install @octokdit/rest nodemailer
+        run: npm install @octokit/rest nodemailer
 
       - name: Run badge automation script
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           EMAIL_USER: ${{ secrets.EMAIL_USER }}
           EMAIL_PASS: ${{ secrets.EMAIL_PASS }}
-        run: |
-          node .github/scripts/badge-automation.js
+        run: node .github/scripts/badge-automation.js