Skip to content

Conversation

@Luisgustavom1
Copy link
Collaborator

@Luisgustavom1 Luisgustavom1 commented Apr 5, 2025

  • add "remove all"
  • Add check no filtros que já estão selecionados
  • Ta puxando o /all do font-awesome - Acho que se criar um kit com o email da tlc da pra pegar só os icones e não puxar coisas desnecessárias
  • Dar um jeito de minificar o html final

@Luisgustavom1 Luisgustavom1 requested a review from wcalderipe April 5, 2025 18:09
@Luisgustavom1 Luisgustavom1 marked this pull request as draft April 5, 2025 18:09
@Luisgustavom1 Luisgustavom1 force-pushed the feat/export-catalog-yaml-to-a-static-page branch from 609f9a7 to 57a09c7 Compare April 6, 2025 02:49
Copy link
Contributor

@wcalderipe wcalderipe left a comment

Choose a reason for hiding this comment

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

Mandou muito bem @Luisgustavom1, valeu pelo PR!

Deixei uns comentários com sugestões na UI e no processo de build. Também abri o PR #27 que adiciona live reload no teu PR com air — pode dar merge se tiver tudo certo pra você.

fmt.Println("error writing public/index.html", err)
os.Exit(1)
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Estava pensando que talvez a gente não precise versionar o build do index.html, igual não se salva o dist num projeto React, por exemplo.

Imaginando o CI como algo BEM simples:

  • Fazer o build do index.html e dos assets dentro da pasta build/site
  • Copiar os arquivos de public pra build/site
  • Fazer o upload de tudo que tá em build/site pra CDN (ex: Cloudflare Pages)

@Luisgustavom1 o que você acha?

Copy link
Contributor

Choose a reason for hiding this comment

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

Se você topar essa ideia e aceitar a proposta #27, vai precisar ajustar o make site/serve pra apontar o servidor pra build/site.

Ah, e lembra de adicionar a pasta build no .gitignore também 😉

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Toppp, eu nunca usei o cloudflare pages, mas a ideia de deixar o build no git era só de apontar pra esse mesmo repo, tipo como a vercel ou o github pages faz, ai não precisaria de CI aqui, por que ele já escuta a branch do repo.

Bom de configurar a CI e subir pra CDN é que a gente já ganha a automação de gerar o readme + site.

Copy link
Collaborator Author

@Luisgustavom1 Luisgustavom1 Apr 13, 2025

Choose a reason for hiding this comment

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

@wcalderipe ficou assim

  1. Apaga a build/site
  2. Criar a build/site
  3. Copia o que ta na public/
  4. Escreve o html renderizado no build/site

@Luisgustavom1 Luisgustavom1 force-pushed the feat/export-catalog-yaml-to-a-static-page branch from 0fb8cbd to e00a229 Compare April 8, 2025 11:48
@Luisgustavom1 Luisgustavom1 force-pushed the feat/export-catalog-yaml-to-a-static-page branch from e6030fd to 4ec9b50 Compare April 9, 2025 00:28
@Luisgustavom1 Luisgustavom1 force-pushed the feat/export-catalog-yaml-to-a-static-page branch from 19ae31c to 1560153 Compare April 13, 2025 13:52
@Luisgustavom1 Luisgustavom1 force-pushed the feat/export-catalog-yaml-to-a-static-page branch from 1560153 to 85d3b30 Compare April 13, 2025 14:05
@Luisgustavom1 Luisgustavom1 marked this pull request as ready for review April 13, 2025 14:05
Copy link
Contributor

@wcalderipe wcalderipe left a comment

Choose a reason for hiding this comment

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

Bom demais @Luisgustavom1!

Deixei duas sugestões na UI mas acho que podemos fazer o merge assim que tu quiser.

Em seguida da para jogar #29 e atualizar o README e o site do repositório com a URL de produção 🚀

image

Comment on lines 157 to 162
<article class="text-center mb-12">
<h1 class="text-3xl font-bold text-gray-900">Awesome Tech Lead</h1>
<p class="mt-2 text-lg text-gray-600">Lista de conteúdo sobre lideraça técnica curada pelos membros da comunidade
<a class="text-blue-500" href="https://techleads.club?utm_source=awesome-tech-lead&utm_medium=site" target="_blank">TechLeads.club</a> 💎
</p>
</article>
Copy link
Contributor

Choose a reason for hiding this comment

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

Como o site vai acabar sendo a principal maneira que vamos falar da lista, uma sugestão é colocar uma referencia ao repositório no site.

Por exemplo, um badge com a quantidade de estralas.

tech-leads-club/awesome-tech-lead badge

<img alt="GitHub User's stars" src="https://img.shields.io/github/stars/tech-leads-club%2Fawesome-tech-lead">

@Luisgustavom1 Luisgustavom1 merged commit 6f304b5 into main Apr 13, 2025
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.

3 participants