Skip to content

Quality of Life: Makefile#43

Closed
ppastorf wants to merge 3 commits intoSOS-RS:developfrom
ppastorf:develop
Closed

Quality of Life: Makefile#43
ppastorf wants to merge 3 commits intoSOS-RS:developfrom
ppastorf:develop

Conversation

@ppastorf
Copy link

@ppastorf ppastorf commented May 10, 2024

Este PR adiciona um Makefile, com algumas diretivas para desenvolvimento, fazendo referencia aos scripts do node configurados no package.json, e para serem futuramente usadas nas workflows de deploy:

  • Adicão do Makefile
  • Atualização do README.md
  • Atualização do workflow de deploy para usar

Por favor, confiram as diretivas criadas no Makefile, e opinem se tem algo errado / faltando

@ppastorf ppastorf requested a review from rafaelss May 11, 2024 15:55
@vanflux
Copy link

vanflux commented May 13, 2024

Parece só um wrapper dos scripts que já estão configurados no npm, não vejo vantagem visto que além de node e docker tem que ter mais o make instalado.

@CesarOliveira
Copy link

gosto do Make file, ajuda bastante novos devs a startar no projeto, geralmente uso em todos projetos que trabalho profissionalmente, vou testar a branch e se tiver ok deixo meu approve ✅

Copy link

@CesarOliveira CesarOliveira left a comment

Choose a reason for hiding this comment

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

LGTM

@ppastorf ppastorf mentioned this pull request May 15, 2024
@ppastorf
Copy link
Author

Parece só um wrapper dos scripts que já estão configurados no npm, não vejo vantagem visto que além de node e docker tem que ter mais o make instalado.

algumas diretivas vao ser usadas na a pipeline de deploy para facilitar

@vanflux
Copy link

vanflux commented May 17, 2024

Parece só um wrapper dos scripts que já estão configurados no npm, não vejo vantagem visto que além de node e docker tem que ter mais o make instalado.

algumas diretivas vao ser usadas na a pipeline de deploy para facilitar

Na minha visão só traz complexidade pro flow das coisas. Quando der problema, ta, mas oq o script sla oq do makefile faz? ah ele tem tal dependencia, vamos ver lá oq é... ah mais isso, mais aquilo. Mas posso estar errado.

@ppastorf ppastorf deleted the branch SOS-RS:develop May 18, 2024 23:04
@ppastorf ppastorf closed this May 18, 2024
@ppastorf ppastorf deleted the develop branch May 18, 2024 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants