Nodemecu/nodemecu
5
0
Fork 0
Code Issues 3 Pull requests 3 Projects Releases Wiki Activity

Arreglos varios #6

Open
fauno wants to merge 21 commits from arreglos into master
pull from: arreglos
merge into: Nodemecu:master
Conversation 0 Commits 21 Files changed 5 +44 -47
fauno commented 2022-03-03 18:52:42 +00:00
Owner
Copy link

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
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.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin arreglos:arreglos
git checkout arreglos

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master
git merge --no-ff arreglos
git checkout arreglos
git rebase master
git checkout master
git merge --ff-only arreglos
git checkout arreglos
git rebase master
git checkout master
git merge --no-ff arreglos
git checkout master
git merge --squash arreglos
git checkout master
git merge --ff-only arreglos
git checkout master
git merge arreglos
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
sergio.pernas
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
sergio.pernas
fauno
1 participant
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.
Delete branch "arreglos"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?