Skip to content

adicionando código de conduta (#56)#61

Merged
vitormattos merged 3 commits intoPHPRio:masterfrom
Cristianpl4y:master
Sep 19, 2024
Merged

adicionando código de conduta (#56)#61
vitormattos merged 3 commits intoPHPRio:masterfrom
Cristianpl4y:master

Conversation

@Cristianpl4y
Copy link
Copy Markdown
Contributor

Olá, equipe do PHP Rio!

Espero que este pull request ajude a melhorar o erro da pagina 404 para o código de conduta. Fiz esta contribuição com base nas boas práticas e na filosofia do projeto, e acredito que ela pode agregar valor à comunidade.

Fico à disposição para ajustes ou esclarecimentos e espero poder colaborar mais com o projeto no futuro.

Agradeço pela oportunidade e por todo o trabalho dedicado ao PHP Rio!

Atenciosamente,
Cristian Pessanha

Copy link
Copy Markdown
Member

@vitormattos vitormattos left a comment

Choose a reason for hiding this comment

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

Teu PR trouxe uma luz...

Olhando no código já estava implementado como git submodule apontando para aquele repo que comentei. O que provavelmente faltou para não dar 404 foi iniciar o git submodule. Dando uma olhada na Action de checkout, ela já tem recurso para inicializar o submodule, podemos testá-lo (https://github.com/actions/checkout). Provavelmente o fix para a issue #56 será ativar para fazer o init do submodule durante o processo de build.

@Cristianpl4y
Copy link
Copy Markdown
Contributor Author

@vitormattos obrigado pelo review! 🚀

Realizei o revert do commit inicial e fiz as devidas alterações no arquivo deploy.yml:

  • No passo de checkout do repositório, atualizei a versão da action de checkout de actions/checkout@v2 para actions/checkout@v3 e adicionei as opções submodules: true e fetch-depth: 0.
  • Com essas mudanças, garantimos que o repositório será clonado corretamente com os submódulos, e todo o histórico será baixado, evitando problemas com referências incompletas de commits.

Agradeço pela oportunidade! Tamo junto! ✌️

@vitormattos vitormattos merged commit 9d988e4 into PHPRio:master Sep 19, 2024
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