Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aggiunge capitolo sviluppatore senior #182

Merged
merged 48 commits into from Feb 19, 2024
Merged

Aggiunge capitolo sviluppatore senior #182

merged 48 commits into from Feb 19, 2024

Conversation

lmammino
Copy link
Contributor

@lmammino lmammino commented Dec 8, 2023

Prima draft del capitolo sul ruolo e le competenze dello sviluppatore senior.

Sfogatevi con i commenti 😀

docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
@Cadienvan
Copy link
Member

Ciao @lmammino ,
grazie mille, capitolo spettacolare!

Ho lasciato giusto alcuni suggerimenti e idee oltre che correzioni di typo nella prima metà del capitolo, è molto lungo e fare una review dall'inizio alla fine in una sola volta è un po' complesso ahah

Mi è piaciuto molto quello che ho letto, soprattutto questo tratto:

Bisogna conoscere le buone pratice come i principi del "Clean code" ma é anche importante non essere dogmatici, ovvero applicare tutti i design patterns e le buone pratice solo perché possiamo farlo. Bisogna piuttosto chiedersi: "qual é il vantaggio di applicare uno specifico pattern ad un dato problema?". Oppure "qual é il costo in termini di complessitá aggiuntiva?".

Sono estremamente d'accordo sulla necessità di uscire dal dogmatismo perché in determinati contesti la complessità di leggibilità e manutenzione non vale le performance o la scalabilità vantata da certe scelte.

@Cadienvan Cadienvan requested a review from a team December 9, 2023 09:31
lmammino and others added 17 commits December 9, 2023 11:32
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
Co-authored-by: Michael Di Prisco <Cadienvan@users.noreply.github.com>
@lmammino
Copy link
Contributor Author

lmammino commented Dec 9, 2023

Grazie mille per la revisione super approfondita, @Cadienvan! Ho fatto mille commit perché non potevo applicare i suggerimenti in batch, ma spero non sia un problema e che potremo fare uno squash prima del merge.

Mi fa piacere che il capitolo ti sia piaciuto e aspetto altre reviews! Siate spietati! 😆

Cadienvan
Cadienvan previously approved these changes Dec 18, 2023
Mik23mik
Mik23mik previously approved these changes Dec 18, 2023
Copy link
Contributor

@allevo allevo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mi è piaciuto molto leggerlo: grande!
Ho messo qualche suggerimento quà e là.
Rivedrei lo spazio tra i paragrafi: personalmente non lo metterei in molti casi, ma è un parere personale.

docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
Co-authored-by: Tommaso Allevi <tomallevi@gmail.com>
@lmammino lmammino dismissed stale reviews from Mik23mik and Cadienvan via 54ed362 December 29, 2023 15:04
Mik23mik
Mik23mik previously approved these changes Feb 10, 2024
Copy link
Member

@corradopetrelli corradopetrelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capitolo bomba! L'unica cosa che non comprendo è perchè la preview di Github non metta l'H3 solo su questa sezione.
image

Anche l'highlight lato codice non lo riconosce
image

😢

@Cadienvan
Copy link
Member

Cominciamo ad arrivare a 6 approvals, quando ci siamo lo fixo io quando mergio!

@nicolaerario
Copy link
Member

@lmammino informazione forse inutile e non richiesta: usi il layout internazionale per la tastiera per caso? (io si btw).
Chiedo perché leggo velocitá, cioé, cosí e, posso sbagliare, credo debbano essere accenti gravi (velocità).

docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
@nicolaerario
Copy link
Member

nicolaerario commented Feb 12, 2024

Lascio dei commentini qui e lì, proseguo successivamente.
Grande il riferimento a tiangolo, azzeccatissimo @lmammino

docs/it/senior.md Outdated Show resolved Hide resolved

Questo comporta sviluppare tutta una serie di competenze quali: comprendere ed approfondire i resoconti degli utenti, scrivere diversi tipi di test (unitari, di integrazione, end-to-end), individuare ed analizzare i casi limite, strutturare i log in modo che siano più facili da leggere e ricercare in caso di errori, riuscire a leggere ed interpretare varie metriche e dashboard.

Un senior dovrebbe essere in grado di fare una grande differenza e di aiutare l’intero team a scrivere un software migliore, illustrando ed educando alle migliori pratiche per il test del software.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La butto lì: linkone al capitolo sul testing? @Cadienvan

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nicolaerario ci potrebbe stare!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'é un modo ben preciso per creare link tra capitoli?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metodo classico dei link con path relativo (vscode dovrebbe dare anche una mano).
Direi: [testo del link](testing.md) , dovrebbe andar bene;
Dovessimo linkare una risorsa in diversa lingua (ad esempio un link ad una pagina che, per assurdo, abbiamo solo in inglese), sarebbe: [testo del link](../en/testing.md)

docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
Copy link
Member

@nicolaerario nicolaerario left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Richiedo cambiamenti solo per i typo; il resto sono tutti commenti opinabili. (vale sia per quelli sopra che sotto)

docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
docs/it/senior.md Outdated Show resolved Hide resolved
@nicolaerario
Copy link
Member

Posso dire un ottimo capitolo. Ora ho qualche spunto in più su come seniorizzarmi senza basarmi solo sulla gradazione di bianco della mia barba.
Credo inoltre che, se il capitolo venisse letto dall'1% dei senior presenti su LinkedIn, avremmo moltissime headline così: Senior software developer.

Grazie @lmammino
grazie ragazzi

@lmammino
Copy link
Contributor Author

Grazie mille @nicolaerario per aver trovato altro margine di miglioramento :) Stiamo limando questo capitolo per benino ma sono sicuro che ne varrà la pena!

PS: ci sono dei punti su cui ho lasciato dei commentini @Cadienvan + @nicolaerario, fatemi sapere che ne pensate!

@nicolaerario
Copy link
Member

nicolaerario commented Feb 19, 2024

Ho risolto la questione "Barbapapà developer" perchè irrilevante.
Ho dato hint sulla creazione del link alla pagina del testing.

Ri-chiedetemi la review che portiamo in prod

Cadienvan
Cadienvan previously approved these changes Feb 19, 2024
docs/it/senior.md Outdated Show resolved Hide resolved
@Cadienvan Cadienvan merged commit 10bd0e2 into main Feb 19, 2024
1 check passed
@Cadienvan Cadienvan deleted the senior-chapter branch February 19, 2024 08:58
@lmammino
Copy link
Contributor Author

🎉🚀

@Cadienvan Cadienvan linked an issue Feb 19, 2024 that may be closed by this pull request
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.

[🆕]: Carriera - Senior
8 participants