@eui/tools 6.1.1 → 6.1.2

package/.version.properties

@@ -1 +1 @@
-6.1.1
+6.1.2

package/CHANGELOG.md

@@ -1,3 +1,13 @@
+## 6.1.2 (2022-11-06)
+
+##### Bug Fixes
+
+* **other:**
+  * slack not rendering certain emoji - MWP-6448 [MWP-6448](https://webgate.ec.europa.eu/CITnet/jira/browse/MWP-6448)  ([ec2cdeb7](https://webgate.ec.europa.eu/CITnet/stash/scm/csdr/eui-tools.git/commits/ec2cdeb7f6d506b40649e8ab98952b7eb0203e46))
+  * styles auditing only for UI packages - EUI-6448 [EUI-6448](https://webgate.ec.europa.eu/CITnet/jira/browse/EUI-6448)  ([265671d9](https://webgate.ec.europa.eu/CITnet/stash/scm/csdr/eui-tools.git/commits/265671d9c756e84771fc2df61d5993957246a268))
+
+* * *
+* * *
 ## 6.1.1 (2022-11-06)
 
 ##### Chores

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@eui/tools",
-    "version": "6.1.1",
+    "version": "6.1.2",
     "tag": "latest",
     "license": "EUPL-1.1",
     "description": "eUI common tools and scripts",

package/scripts/csdr/audit/styles.js

@@ -337,79 +337,86 @@ const runStylesAudit = module.exports.runStylesAudit = (pkg) => {
 
  // CRITICAL ONLY
 
- report.finalReport['inlineStyles'] = {
-   description: 'inline styles found in HTML',
-   value: report.inlineStyles.count,
-   status: (report.inlineStyles.count !== 0) ? 'CRITICAL' : 'OK',
-   comment: (report.inlineStyles.count !== 0) ? 'No inline styles should be present in html, usage classes with scss / eUI utilities instead' : null
- }
- report.finalReport['material'] = {
-   description: 'material component usage found in HTML',
-   value: report.material.count,
-   status: (report.material.count !== 0) ? 'CRITICAL' : 'OK',
-   comment: (report.material.count !== 0) ? 'Usage of material should be avoided' : null
- }
- report.finalReport['primeng'] = {
-   description: 'PrimeNG components usage found in HTML',
-   value: report.primeng.count,
-   status: (report.primeng.count !== 0) ? 'CRITICAL' : 'OK',
-   comment: (report.primeng.count !== 0) ? 'Remove any PrimeNG usage, deprecated in eUI' : null
- }
-
+ if (report.htmlLinesCount !== 0) {
+   report.finalReport['inlineStyles'] = {
+     description: 'inline styles found in HTML',
+     value: report.inlineStyles.count,
+     status: (report.inlineStyles.count !== 0) ? 'CRITICAL' : 'OK',
+     comment: (report.inlineStyles.count !== 0) ? 'No inline styles should be present in html, usage classes with scss / eUI utilities instead' : null
+   }
+   report.finalReport['material'] = {
+     description: 'material component usage found in HTML',
+     value: report.material.count,
+     status: (report.material.count !== 0) ? 'CRITICAL' : 'OK',
+     comment: (report.material.count !== 0) ? 'Usage of material should be avoided' : null
+   }
+   report.finalReport['primeng'] = {
+     description: 'PrimeNG components usage found in HTML',
+     value: report.primeng.count,
+     status: (report.primeng.count !== 0) ? 'CRITICAL' : 'OK',
+     comment: (report.primeng.count !== 0) ? 'Remove any PrimeNG usage, deprecated in eUI' : null
+   }
 
+  // IMPROVE ONLY
 
- // IMPROVE ONLY
+  report.finalReport['uxCmp'] = {
+    description: 'eUI ux- components usage found in HTML',
+    value: report.uxCmp.count,
+    status: (report.uxCmp.count !== 0) ? 'IMPROVE' : 'OK',
+    comment: (report.uxCmp.count !== 0) ? 'Replace by "eui-" components whenever possible, to ensure future proof migration' : null
+  }
 
- report.finalReport['uxCmp'] = {
-   description: 'eUI ux- components usage found in HTML',
-   value: report.uxCmp.count,
-   status: (report.uxCmp.count !== 0) ? 'IMPROVE' : 'OK',
-   comment: (report.uxCmp.count !== 0) ? 'Replace by "eui-" components whenever possible, to ensure future proof migration' : null
+  report.finalReport['tsOver500'] = {
+    description: 'TS file containing more than 500 lines',
+    value: report.tsOver500.count,
+    status: (report.tsOver500.count !== 0) ? 'IMPROVE' : 'OK',
+    comment: (report.tsOver500.count !== 0) ? 'Split TS files into multiple ones for readability' : null
+  }
  }
 
- report.finalReport['tsOver500'] = {
-   description: 'TS file containing more than 500 lines',
-   value: report.tsOver500.count,
-   status: (report.tsOver500.count !== 0) ? 'IMPROVE' : 'OK',
-   comment: (report.tsOver500.count !== 0) ? 'Split TS files into multiple ones for readability' : null
- }
 
 
- // Calculating score
+ if (Object.keys(report.finalReport).length === 0) {
+  tools.logInfo('No sources to audit...skipping');
+  return null;
 
- let score = 0, total = 0;
+ } else {
+   // Calculating score
 
- Object.entries(report.finalReport).forEach((v) => {
-  total++;
+   let score = 0, total = 0;
 
-  switch(v[1].status) {
-    case 'OK':
-      score++;
-      break;
-    case 'IMPROVE':
-      score+=0.5;
-      break;
-  }
- });
+   Object.entries(report.finalReport).forEach((v) => {
+    total++;
+
+    switch(v[1].status) {
+      case 'OK':
+        score++;
+        break;
+      case 'IMPROVE':
+        score+=0.5;
+        break;
+    }
+   });
 
- report.score = score;
- report.total = total;
- report.pcScore = Math.floor(100 * score / total);
+   report.score = score;
+   report.total = total;
+   report.pcScore = Math.floor(100 * score / total);
 
 
- // return JSON report
+   // return JSON report
 
- tools.logInfo('STATS report');
- console.log(JSON.stringify(report, null, 2));
+   tools.logInfo('STATS report');
+   console.log(JSON.stringify(report, null, 2));
 
- return report;
+   return report;
+ }
 }
 
 
 
 
 module.exports.audit = (pkg) => {
-  if (pkg.remote) {
+  if (pkg.remote && pkg.name.indexOf('-ui') === -1) {
     return;
   }
 
@@ -420,6 +427,11 @@ module.exports.audit = (pkg) => {
 
     const report = runStylesAudit(pkg);
 
+    if (!report) {
+      return;
+    }
+
+
     // FINAL REPORT
 
     let finalReportMessage = `*STYLES AUDIT REPORT for ${pkg.name}:*\n\n`;
@@ -432,10 +444,10 @@ module.exports.audit = (pkg) => {
       } else if (v[1].status === 'IMPROVE') {
         statusEmoji = ':warning:';
       } else {
-        statusEmoji = ':white_tick:';
+        statusEmoji = ':large_green_circle:';
       }
 
-      finalReportMessage += `:black_small_square: ${statusEmoji} *${v[1].description}*: found *${v[1].value}* - status: *${v[1].status}* ${(v[1].comment) ? '==> ' + v[1].comment : ''}\n`;
+      finalReportMessage += `${statusEmoji} *${v[1].description}*: found *${v[1].value}* - status: *${v[1].status}* ${(v[1].comment) ? '==> ' + v[1].comment : ''}\n`;
     });
 
     finalReportMessage +=`\neUI compliance score : *${report.score}* out of *${report.total}* *(${report.pcScore}%)*`;