Skip to content

Update gulp and related libraries#1577

Merged
loviuz merged 2 commits intodevcode-it:masterfrom
Dasc3er:npm-update
May 1, 2025
Merged

Update gulp and related libraries#1577
loviuz merged 2 commits intodevcode-it:masterfrom
Dasc3er:npm-update

Conversation

@Dasc3er
Copy link
Copy Markdown
Contributor

@Dasc3er Dasc3er commented Jan 11, 2025

Aggiornamento di gulp e librerie collegate.
Da testare per verificare che tutti gli assets siano compilati correttamente (Font Awesome sta dando alcuni problemi in locale).

Tipologia

  • Minor change

Checklist

  • Il codice segue le linee guida del progetto
  • Il codice non genera warnings

@Dasc3er Dasc3er marked this pull request as ready for review January 11, 2025 16:30
@loviuz
Copy link
Copy Markdown
Contributor

loviuz commented Feb 15, 2025

Ciao @Dasc3er , ti risulta si vedano le immagini? Da me, per esempio, assets/dist/img/logo_completo.png è diverso da quello in assets/src/img/, nel senso che se lo apro da browser mi dice che l'immagine ha errori, e in effetti aprendolo con un editor di testo ha dei caratteri un po' diversi tali per cui non lo vede nemmeno come una png stranamente...

@loviuz
Copy link
Copy Markdown
Contributor

loviuz commented Feb 15, 2025

Ok, gulp 5 ha un bug che corrompe le immagini 😅

gulpjs/gulp#2775

Sto testando con {encoding:false} e tentando di capire cos'hanno i fonts...

@Dasc3er
Copy link
Copy Markdown
Contributor Author

Dasc3er commented Feb 15, 2025

Perfetto, questa era una delle cose da verificare - insieme ai font, che sembrano corrotti in modo simile. Ti direi prova la stessa configurazione?

@Dasc3er
Copy link
Copy Markdown
Contributor Author

Dasc3er commented Feb 15, 2025

Sembra read: false sia un altra opzione valida, che dovrebbe anche velocizzare le operazioni di copia

@Dasc3er Dasc3er force-pushed the npm-update branch 2 times, most recently from 18647fd to 1b99ebe Compare February 16, 2025 07:46
@Dasc3er
Copy link
Copy Markdown
Contributor Author

Dasc3er commented Feb 16, 2025

Aggiornato, ho verificato che funzioni correttamente ma un ulteriore verifica sarebbe utile.

Attenzione nella gestione di task gulp che devono essere tutte attese alla fine della funzione (altrimenti il codice potrebbe causare errori random, se non finiscono in contemporanea).

@loviuz loviuz merged commit 1083bbb into devcode-it:master May 1, 2025
3 checks passed
@loviuz loviuz deleted the npm-update branch May 1, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants