Arreglos varios #6

Open

fauno wants to merge 21 commits from arreglos into master

fauno commented

2022-03-03 18:52:42 +00:00

Owner

Resumen:

Había ubicaciones configurables que en los archivos estaban hardcodeadas, así que las cambié a todas por las variables
Hice configurable el linter de JSON porque no tengo jsonlint-php en mi distro
Reordené la lógica de sincronización para que el loop no pinguee más de la cuenta
Había una fracción de timeout en curl en 0,9 que cambié a 1 porque debería ser 0.9 pero curl no aceptaba fracciones de todas formas (entendiendo que era un timeout de 1s no?)
Cree un archivo de fallados para mover todas las sincronizaciones fallidas, pero no sé si tiene sentido
Eliminé algo de código que no se usaba más

Resumen: * Había ubicaciones configurables que en los archivos estaban hardcodeadas, así que las cambié a todas por las variables * Hice configurable el linter de JSON porque no tengo jsonlint-php en mi distro * Reordené la lógica de sincronización para que el loop no pinguee más de la cuenta * Había una fracción de timeout en curl en 0,9 que cambié a 1 porque debería ser 0.9 pero curl no aceptaba fracciones de todas formas (entendiendo que era un timeout de 1s no?) * Cree un archivo de fallados para mover todas las sincronizaciones fallidas, pero no sé si tiene sentido * Eliminé algo de código que no se usaba más

sergio.pernas was assigned by fauno

2022-03-03 18:52:42 +00:00

fauno added 1 commit 2022-03-03 19:09:46 +00:00

d1e44141f6 usar tabs

fauno requested review from sergio.pernas 2022-03-03 19:12:02 +00:00

fauno self-assigned this 2022-03-03 19:12:10 +00:00

This pull request cannot be merged automatically due to conflicts.

Merge manually to resolve the conflicts.

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b arreglos master

git pull origin arreglos

Step 2:

Merge the changes and update on Forgejo.

git checkout master

git merge --no-ff arreglos

git push origin master

No reviewers

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Nodemecu/nodemecu#6

No description provided.