Skip to content

Conversation

luizbrito7
Copy link
Member

Pull Request Form

Caution

Antes de subir o pr faça uma validação do código que foi adicionado nos commits enviados!

Checklist

  • Meu código segue as diretrizes do projeto.
  • Executei testes locais e eles passaram.
  • Documentação atualizada (se aplicável).

Descrição da PR (descreva o que foi feito de forma objetiva):

Essa PR implementa no repositório o script para criar repos e ruleset baseado em um template, facilitando assim as configs iniciais necessárias para o git flow

Screenshots dos testes (se aplicável)

image

Observações Adicionais

Posteriormente o script vai evoluir para uma pipeline com o avanço do projeto.

Links importantes:

2. E necessário uma PR antes de fazer o merge com as seguintes propriedades:
- 1 aprovação é necessária para seguir com merge
- E necessário que as conversas sejam solucionadas
- O copilot vai realizar code review (influencia no 2.2)
Copy link
Member

Choose a reason for hiding this comment

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

Por favor, esclarecer este trecho "(influência no 2.2)", revisar funcionamento do copilot


O objetivo desse script é criar um repositório no Github e configurar as rulesets abaixo:

1. Proibido deletar as branchs de referencia (develop e production)
Copy link
Member

Choose a reason for hiding this comment

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

Referência

O objetivo desse script é criar um repositório no Github e configurar as rulesets abaixo:

1. Proibido deletar as branchs de referencia (develop e production)
2. E necessário uma PR antes de fazer o merge com as seguintes propriedades:
Copy link
Member

Choose a reason for hiding this comment

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

É necessário

gh repo create ${org_name}/${repo_name} --public --template ${org_name}/${template} --include-all-branches
```

O comando acima para criar o repositório precisa do gh cli instalado e autenticado!
Copy link
Member

Choose a reason for hiding this comment

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

Rever frase

Copy link
Member

@MatthewsTomts MatthewsTomts left a comment

Choose a reason for hiding this comment

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

Por favor, revise os comentários, corrigir o "danger" no repositório do template e o funcionamento do copilot

@luizbrito7
Copy link
Member Author

luizbrito7 commented Jun 21, 2025

Por favor, revise os comentários, corrigir o "danger" no repositório do template e o funcionamento do copilot

project-alphapet/devops-template-repo#1 PR da alteração danger para caution no pull_request_template.md

@luizbrito7 luizbrito7 removed the request for review from pwnkcin June 21, 2025 21:34
@luizbrito7
Copy link
Member Author

@MatthewsTomts tive que remover o code review com o copilot, o recurso não está disponível em nível gratuito:

image

@luizbrito7 luizbrito7 requested a review from MatthewsTomts June 21, 2025 21:50
@luizbrito7 luizbrito7 requested a review from Copilot July 8, 2025 23:17
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a shell script to automate GitHub repository creation (with a template) and apply branch protection rulesets, along with accompanying documentation.

  • Adds script-create-repo.sh to create repos and configure a branch ruleset via gh CLI and curl
  • Provides a README.md detailing required environment variables and script behavior

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
script-create-repo/script-create-repo.sh New script for repo creation and ruleset configuration
script-create-repo/README.md Documentation for setting up and running the script
Comments suppressed due to low confidence (2)

script-create-repo/script-create-repo.sh:1

  • [nitpick] Include a brief header or comment block describing script purpose, usage syntax, and prerequisite tools.
# envs 

script-create-repo/script-create-repo.sh:1

  • [nitpick] Consider adding automated tests or a dry-run mode to verify script logic without side effects.
# envs 

template="${4}"

# create repo
gh repo create ${org_name}/${repo_name} --public --template ${org_name}/${template} --include-all-branches
Copy link

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

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

Wrap variable expansions in quotes to prevent word splitting (e.g., "${org_name}/${repo_name}").

Suggested change
gh repo create ${org_name}/${repo_name} --public --template ${org_name}/${template} --include-all-branches
gh repo create "${org_name}/${repo_name}" --public --template "${org_name}/${template}" --include-all-branches

Copilot uses AI. Check for mistakes.

@@ -0,0 +1,53 @@
# envs
repo_name="${1}"
Copy link

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

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

Add validation for required arguments at the top of the script and print a usage message if any are missing.

Copilot uses AI. Check for mistakes.

Comment on lines +6 to +11
1. Proibido deletar as branchs de referência (develop e main)
2. E necessário uma PR antes de fazer o merge com as seguintes propriedades:
- 1 aprovação é necessária para seguir com merge
- É necessário que as conversas sejam solucionadas
- `Squash` e `rebase` não são aceitos somente`merge`
3. Push direto nas branchs de referência são bloqueados
Copy link

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

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

Correct spelling: change "branchs" to "branches".

Suggested change
1. Proibido deletar as branchs de referência (develop e main)
2. E necessário uma PR antes de fazer o merge com as seguintes propriedades:
- 1 aprovação é necessária para seguir com merge
- É necessário que as conversas sejam solucionadas
- `Squash` e `rebase` não são aceitos somente`merge`
3. Push direto nas branchs de referência são bloqueados
1. Proibido deletar as branches de referência (develop e main)
2. E necessário uma PR antes de fazer o merge com as seguintes propriedades:
- 1 aprovação é necessária para seguir com merge
- É necessário que as conversas sejam solucionadas
- `Squash` e `rebase` não são aceitos somente`merge`
3. Push direto nas branches de referência são bloqueados

Copilot uses AI. Check for mistakes.

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