(zeppelin-site) branch master updated: [HOTFIX] Fix Mapping Issues in Zeppelin Visualization Dependencies (#21)

[jongyoul]

This is an automated email from the ASF dual-hosted git repository.

jongyoul pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin-site.git


The following commit(s) were added to refs/heads/master by this push:
     new 4fe103ef3 [HOTFIX] Fix Mapping Issues in Zeppelin Visualization 
Dependencies (#21)
4fe103ef3 is described below

commit 4fe103ef3c342085b58b512daedd90968a9c2f82
Author: pythonstrup <pythonst...@gmail.com>
AuthorDate: Sun Sep 29 22:24:41 2024 +0900

    [HOTFIX] Fix Mapping Issues in Zeppelin Visualization Dependencies (#21)
---
 .github/workflows/helium/lib/fetchHeliumPkgInfo.js | 51 +++++++++-------------
 1 file changed, 21 insertions(+), 30 deletions(-)

diff --git a/.github/workflows/helium/lib/fetchHeliumPkgInfo.js 
b/.github/workflows/helium/lib/fetchHeliumPkgInfo.js
index 39761f1a4..06a8052bd 100644
--- a/.github/workflows/helium/lib/fetchHeliumPkgInfo.js
+++ b/.github/workflows/helium/lib/fetchHeliumPkgInfo.js
@@ -5,13 +5,6 @@ import dependants from 'npm-dependants';
 
 const depList = ['zeppelin-vis', 'zeppelin-spell'];
 
-const removeExtraSquareBracket = function (jsonArray) {
-  jsonArray = jsonArray.map(function (e) {
-    return stringify(e);
-  });
-  return jsonArray.join(',');
-};
-
 const setEachVersionInfo = function (dependency, data) {
   const eachVerInfo = {};
   const versions = data.versions;
@@ -33,18 +26,18 @@ const setEachVersionInfo = function (dependency, data) {
       published: data.time[verTag],
       artifact: key._id,
       author:
-        key.author?.name == undefined && key.maintainers[0].name == undefined
-          ? 'unknown'
-          : key.author?.name || key.maintainers[0].name,
+          key.author?.name == undefined && key.maintainers[0].name == undefined
+              ? 'unknown'
+              : key.author?.name || key.maintainers[0].name,
       description: key.description == undefined ? 'unknown' : key.description,
       license: key.license == undefined ? 'unknown' : key.license,
       icon: key.helium
-        ? key.helium.icon == undefined
-          ? defaultIcon
-          : key.helium.icon
-        : key.icon == undefined
-        ? defaultIcon
-        : key.icon,
+          ? key.helium.icon == undefined
+              ? defaultIcon
+              : key.helium.icon
+          : key.icon == undefined
+              ? defaultIcon
+              : key.icon,
       config: key.helium ? key.helium.config : undefined,
       spell: key.helium ? key.helium.spell : undefined,
     };
@@ -54,20 +47,18 @@ const setEachVersionInfo = function (dependency, data) {
 
 const finalResult = {};
 const client = new RegistryClient();
-async function getDependencies(packageName) {
+async function getDependencies(packageName, dependency) {
   await client.get(
-    `https://registry.npmjs.org/${packageName}`,
-    { timeout: 1000 },
-    (error, data) => {
-      if (error) {
-        console.error(error.message);
-        return reject(error);
-      }
+      `https://registry.npmjs.org/${packageName}`,
+      { timeout: 1000 },
+      (error, data) => {
+        if (error) {
+          console.error(error.message);
+          return reject(error);
+        }
 
-      const pkgInfo = {};
-      finalResult[data.name] = setEachVersionInfo(packageName, data);
-      // var result = removeExtraSquareBracket(finalResult).replace(/]|[[]/g, 
'').trim()
-    }
+        finalResult[data.name] = setEachVersionInfo(dependency, data);
+      }
   );
 }
 
@@ -75,9 +66,9 @@ const delay = (timeToDelay) => new Promise((resolve) => 
setTimeout(resolve, time
 export async function fetchHelium() {
   depList.map(async function (dependency) {
     for await (const dependant of dependants(dependency)) {
-      await getDependencies(dependant);
+      await getDependencies(dependant, dependency);
     }
   });
-  await delay(10000);
+  await delay(12000);
   return finalResult;
 }