qa-control 0.6.10 → 0.6.12

package/.github/workflows/build-and-test.yml

@@ -0,0 +1,11 @@
+name: Build and test
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+    branches: [master]
+
+jobs:
+  ci:
+    uses: codenautas/.github/.github/workflows/node-build-and-test.yaml@v1

package/.github/workflows/publish.yml

@@ -0,0 +1,16 @@
+name: Publish from Tag
+
+on:
+  push:
+    tags:
+      - 'v[0-9]+.[0-9]*'
+
+permissions:
+  id-token: write
+  contents: read
+
+jobs:
+  publish:
+    uses: codenautas/.github/.github/workflows/npm-publish-from-tag.yaml@v1
+    with:
+      node_version: '24'

package/.github/workflows/qa-control.yml

@@ -0,0 +1,11 @@
+name: QA control
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+    branches: [master]
+
+jobs:
+  qa:
+    uses: codenautas/.github/.github/workflows/qa-control.yaml@v1

package/bin/definition/definition.js

@@ -332,7 +332,7 @@ module.exports = function(qaControl){
                             }else{
                                 var observedValue=qaControlSection[sectionName];
                                 if(sectionDef.values && !(observedValue in sectionDef.values)){
-                                    warns.push({warning:'invalid_value_1_in_parameter_2',params:[observedValue,sectionName], scoring:{warnings:1}});
+                                    warns.push({warning:'invalid_value_1_in_parameter_2_valid_values_3',params:[observedValue,sectionName,Object.keys(sectionDef.values).join(', ')], scoring:{warnings:1}});
                                 }
                             }
                         }
@@ -584,7 +584,7 @@ module.exports = function(qaControl){
                             var file=obtainedLangs.langs[lang].fileName;
                             if(file !== defReadme) {
                                 var mlContent = multilang.changeNamedDoc(file, content, lang);
-                                if(mlContent !== info.files[file].content) {
+                                if(qaControl.fixEOL(mlContent) !== qaControl.fixEOL(info.files[file].content)) {
                                     warns.push({warning:'readme_multilang_not_sincronized_with_file_1', params:[file], scoring:{multilang:1}});
                                 }
                             }
@@ -749,7 +749,7 @@ module.exports = function(qaControl){
                             return Promise.all(qaFiles.map(function(fileName) {
                                 return fs.readFile(Path.join(qaWorkflowsDir, fileName), 'utf8').then(function(qaContent) {
                                     return fs.readFile(Path.join(projWorkflowsDir, fileName), 'utf8').then(function(projContent) {
-                                        if(qaContent !== projContent) {
+                                        if(qaControl.fixEOL(qaContent) !== qaControl.fixEOL(projContent)) {
                                             return [{warning:'workflow_file_1_differs', params:[fileName], scoring:{workflows:1}}];
                                         }
                                         return [];

package/bin/qa-control.js

@@ -5,6 +5,7 @@ var qaControl = {};
 
 var fs = require('fs-extra');
 var Path = require('path');
+var os = require('os');
 var stripBom = require('strip-bom-string');
 var yaml = require('js-yaml');
 var semver = require("semver");
@@ -33,7 +34,7 @@ qaControl.msgs={
     },
     es:{
         deprecated_version: 'la version es demasiado vieja',
-        invalid_value_1_in_parameter_2: 'valor invalido "$1" para el parametro "$2" en la sección qa-control',
+        invalid_value_1_in_parameter_2_valid_values_3: 'valor invalido "$1" para el parametro "$2" en la sección qa-control. Valores válidos: $3',
         lack_of_mandatory_file_1: 'falta el archivo obligatorio "$1"',
         //lack_of_mandatory_parameter_1: 'falta el parámetro obligatorio "$1"',
         lack_of_mandatory_section_1: 'falta la sección obligatoria "$1" en la sección qa-control',
@@ -102,7 +103,7 @@ qaControl.cmdMsgs = {
 
 // devuelve un buffer con los \n, \r\n, \r como \n
 qaControl.fixEOL = function fixEOL(buf) {
-    return buf.replace(/\s*\r?\n/g, '\n').replace(/\s*\r/g, '\n');
+    return buf.replace(/[^\S\r\n]*(?:\r\n?|\n)/g, os.EOL);
 };
 
 // bufTest debe empezar con bufStart

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qa-control",
   "description": "QA control of things that nobody controls",
-  "version": "0.6.10",
+  "version": "0.6.12",
   "author": "Codenautas <codenautas@googlegroups.com>",
   "license": "MIT",
   "repository": "codenautas/qa-control",
@@ -59,7 +59,8 @@
     "all": "npm test && npm run test-cov && npm run report && npm run qac"
   },
   "files": [
-    "bin"
+    "bin",
+    ".github/workflows"
   ],
   "qa-control": {
     "package-version": "0.3.0",