search

CastalioPodcast / castaliopodcast.github.io

Um podcast inspirado para castálio!
http://castalio.info
Creative Commons Attribution Share Alike 4.0 International
31 stars 12 forks source link

Converter os show notes para usar a diretiva top5 #163

Closed elyezer closed 3 years ago

elyezer commented 4 years ago

No último episódio publicado foi adicionada a diretiva top5 que permite centralizar os items recomendados no Top 5 e também contar quantas vezes um item específico foi recomendado. Isso foi adicionado com o PR #158.

Já no PR #161, foi criada uma nova diretiva que permitiu criar a página com os items mais recomendados (veja em https://castalio.info/top5.html). Basicamente essa diretiva usa informações da diretiva top5 e cria um ranking dos items mais recomendados.

No PR #162 o arquivo geral com os items do Top 5 foi atualizado e deve prover quase tudo que foi mencionado nos show notes, o que vai facilitar converter todos os show notes para usar a diretiva top5. Também no #142, 5 show notes foram convertidos para usar a nova diretiva, ele serve de exemplo para converter os demais show notes.

Essa issue tem como objetivo capturar o que ainda falta ser convertido e possibilita que mais de uma pessoa contribua com a conversão. Se você pretende ajudar com a conversão, deixa um comentário nessa issue dizendo qual o episódio ou os episódios você pretende converter, dessa forma evitamos trabalho duplicado.

A checklist abaixo mostra o que já foi convertido e revisado e o que falta ser convertido ou revisado:

luckdomingues commented 4 years ago

Bom dia man, vi a tua chamada no twitter, me fala qual o processo para ajudar. Vamos fazer um teste com o " 055-editor-vim.rst" e se eu conseguir desenrolar te mando uma nova lista.

elyezer commented 4 years ago

@luckdomingues muito obrigado pelo interesse. O episódio que você mencionou não tem a sessão Top 5 então esse pode ser considerado como já feito. Eu recomendaria algum entre o 102 e 135 a maioria deles são entrevistas e tem a sessão Top 5.

Pegando um como exemplo aqui, vamos com o https://github.com/CastalioPodcast/castaliopodcast.github.io/blob/pelican/content/episodes/110-willian-molinari.rst

Se você abrir o arquivo no seu editor você vai ver algo como:

Top 5
-----

* **Música:** `Pantera - Mouth of War <https://www.last.fm/pt/music/Pantera/_/Mouth+for+War>`_
* **Música:** `Black Label Society - Demise of Sanity <https://www.last.fm/pt/music/Black+Label+Society/1919+Eternal/Demise+Of+Sanity>`_
* **Música:** `Mega Driver <https://www.last.fm/music/Megadriver>`_
* **Música:** `Andy McKee <https://www.last.fm/pt/music/Andy+McKee>`_
* **Música:** `Jon Gomm <https://www.last.fm/music/Jon+Gomm>`_
* **Filme:** `Mr. Robot <http://www.imdb.com/title/tt4158110/>`_
* **Filme:** `Senhor dos Anéis <http://www.imdb.com/title/tt0120737/>`_
* **Filme:** `Watchmen <http://www.imdb.com/title/tt0409459/>`_
* **Filme:** `Game of Thrones <http://www.imdb.com/title/tt0944947/>`_
* **Filme:** `Sherlock BBC <www.imdb.com/title/tt1475582>`_
* **Filme:** `House of cards <http://www.imdb.com/title/tt1856010/>`_
* **Livro:** `J.R.R Tolkien <http://www.goodreads.com/author/show/656983.J_R_R_Tolkien>`_
* **Livro:** `Deep Work <https://www.goodreads.com/book/show/25744928-deep-work>`_
* **Livro:** `Mastery <http://www.goodreads.com/book/show/13589182-mastery>`_
* **Livro:** `The Pragmatic Programmer <http://www.goodreads.com/book/show/4099.The_Pragmatic_Programmer>`_
* **Livro:** `O Restaurante no fim do universo <http://www.goodreads.com/book/show/10021253-o-restaurante-no-fim-do-universo>`_

Aí você trocaria por algo como:

.. top5::

    :music:
        * Pantera - Mouth of War
        * Black Label Society - Demise of Sanity
        * Mega Driver
        * Andy McKee
        * Jon Gomm
    :movie:
        * Mr. Robot
        * Senhor dos Anéis
        * Watchmen
        * Game of Thrones
        * Sherlock BBC
        * House of cards
    :book:
        * J.R.R Tolkien
        * Deep Work
        * Mastery
        * The Pragmatic Programmer
        * O Restaurante no fim do universo

Observe que a linha em branco entre a primeira e terceira linha são importantes. Alguns episódios não tem o link inline como no exemplo que coloquei, aí seria algo como nesse aqui: https://github.com/CastalioPodcast/castaliopodcast.github.io/pull/162/files#diff-e2dd906957478424d1abe420ce03c4b0.

Eu aproveitei e criei o PR com o exemplo acima caso queira dar uma olhada https://github.com/CastalioPodcast/castaliopodcast.github.io/pull/165

luckdomingues commented 4 years ago

@elyezer escolhe um e me diz q faço um teste. fico no aguardo. #tamojunto

elyezer commented 4 years ago

@luckdomingues pode ser algum entre o 130-134, talvez o 134? Antes de mandar o PR tenha certeza de estar cadastrado no https://hacktoberfest.digitalocean.com para poder contabilizar o PR para vc. Muito obrigado

luckdomingues commented 4 years ago
  1. @elyezer , é obrigatório para o envio estar cadastrado no hacktoberfest?
  2. Fiz o 134, me diz se é isso ?
luckdomingues commented 4 years ago

@elyezer , como ninguém reclamou de nada do q fiz no 134, então estou pegando do 130 ao 133. Segue o plano. ok ?

luckdomingues commented 4 years ago

@elyezer , outra duvida, o que é preciso para resolver isso??? image

luckdomingues commented 4 years ago

@elyezer , como ninguém reclamou de nada do q fiz no 134, então estou pegando do 130 ao 133. Segue o plano. ok ?

Fiz do 130 ao 134 ! :)

elyezer commented 4 years ago

@luckdomingues eu comentei nos PRs para resolver os conflitos mas vou colocar aqui o que normalmente faço:

Quando crio meu fork no github normalmente faço um git clone <url> e com isso tenho o meu fork e eu consigo fazer as mudanças e mandar os PRs. Para evitar conflito é bom, sempre que for mandar um PR, fazer um rebase dos nossos branches locais em cima do branch em desenvolvimento (aqui seria o pelican). Para fazer isso primeiro eu adiciono um novo remote:

git remote add upstream https://github.com/CastalioPodcast/castaliopodcast.github.io.git

Dessa forma eu consigo buscas as últimas mudanças fazendo um:

git fetch upstream

Agora que tenho as mudanças mais recentes eu consigo fazer o rebase, para isso primeiro vou ao branch que quero atualizar, depois faço o rebase e mando as mudanças:

git checkout <nome-branch>
git rebase upstream/devel
git push -f origin <nome-branch>

Vale ressaltar que o git push -f no último comando vai sobrescrever o branch remoto então tenha certeza que esta usando o <nome-branch> certo. Considerando o seguinte branch conversao-top5-episode-133 os comandos seriam assim:

git fetch upstream
git checkout conversao-top5-episode-133
git rebase upstream/devel
git push -f origin conversao-top5-episode-133

Uma dica para evitar ter que fazer rebase muitas vezes seria toda vez que for criar um branch, fazer um fetch no upstream e criar o branch assim:

git checkout -b <nome-do-novo-branch> upstream/<branch-devel-no-upstream>

Dessa forma você cria um branch que já esta com o rebase feito em cima do branch destino no upstream. O que aconteceu com os seus PRs foi que você modificou um episódio, fez o commit, e depois criou o branch sem especificar o branch base então o git por padrão pega o branch atual se vc olhar na lista de commits do PR #170 (https://github.com/CastalioPodcast/castaliopodcast.github.io/pull/170/commits) você vai ver que ele inclui todos os commits que você fez anteriormente. O ideal nesse caso seria ter apenas um commit em cada PR, dessa forma você evita conflitos.

Caso alguma coisa não tenha ficado claro me fala que tento colocar mais exemplos ou explicar melhor.

HigorMonteiro commented 4 years ago

@elyezer gostária de contribuir inicialmente com 007, 008 e 009 :)

HigorMonteiro commented 4 years ago

@elyezer acabei ficando empolgado e fiz os show notes do 007 a 017 ;)

luckdomingues commented 4 years ago

@elyezer , reserva do 121 ao 129 agora

elyezer commented 4 years ago

@luckdomingues durante o fim de semana tivemos algum PRs e agora tudo do 100 ao 143 estão completos. Eu atualizei a lista da descrição com o que já está feito. Nem todos que faltam tem o top 5 então estou tentando passar por eles e já ir marcando.

HigorMonteiro commented 4 years ago

@elyezer o checklist está atualizado?

elyezer commented 4 years ago

@HigorMonteiro esta sim, estou trabalhando nos 20-29 atualmente.

elyssonmr commented 3 years ago

Vou trabalhar no 31 ao 35 :)

elyezer commented 3 years ago

Com o commit f98a1f20fe02cdd6aaf01d5cfa66799d072359dd a conversao esta agora completa.