verdnatura/myt
verdnatura
/
myt
0
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

6204-fixturesByEnvironment #2

Merged
jgallego merged 7 commits from 6204-fixturesByEnvironment into master 2023-10-30 06:57:37 +00:00
Conversation 1 Commits 7 Files Changed 5 +18
1 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit b503cbfa9f - Show all commits

18
myt-push.js
View File

@ -151,6 +151,21 @@ class Push extends Command {
function isUndoScript(script) {
return /\.undo\.sql$/.test(script);
}
function isOtherEnvScript(script, env) {
jgallego marked this conversation as resolved Outdated
juan commented 2023-09-26 06:56:45 +00:00
Outdated
Review
Copy Link

No es necesario declarar una función, el código solo se utiliza una vez

No es necesario declarar una función, el código solo se utiliza una vez
jgallego commented 2023-09-26 07:58:39 +00:00
Outdated
Review
Copy Link

Cierto te paso una propuesta de como queda sin funcion, pero creo que es bastante menos legible

// Reemplazo de la función isOtherRealmScript
const splitScript = script.split('.');
const realmPart = splitScript[splitScript.length - 2];

if (splitScript.length > 2) {
    if (!realm && realmPart) {
        continue;
    }
    
    if (realmPart && realmPart !== realm) {
        continue;
    }
}
Cierto te paso una propuesta de como queda sin funcion, pero creo que es bastante menos legible ``` // Reemplazo de la función isOtherRealmScript const splitScript = script.split('.'); const realmPart = splitScript[splitScript.length - 2]; if (splitScript.length > 2) { if (!realm && realmPart) { continue; } if (realmPart && realmPart !== realm) { continue; } } ```
const splitScript = script.split('.');
const envPart = splitScript[splitScript.length - 2];
if (splitScript.length <= 2) {
return false;
}
if (!env) {
return !!envPart;
}
return envPart && envPart !== env;
}
const skipFiles = new Set([
'README.md',
@ -210,6 +225,9 @@ class Push extends Command {
if (isUndoScript(script))
continue;
if (isOtherEnvScript(script, opts.env))
continue;
const [[row]] = await conn.query(
`SELECT errorNumber FROM versionLog
WHERE code = ?