Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refino do arquivo .gitignore #23

Merged
merged 7 commits into from
Mar 14, 2021
Merged

Refino do arquivo .gitignore #23

merged 7 commits into from
Mar 14, 2021

Conversation

fazedordecodigo
Copy link
Collaborator

#20

@jonathanmach
Copy link
Collaborator

Hey! Vi que tem várias regras de extensões que eu nem nunca ouvi falar - qual a ideia de adicionarmos essas regras para casos que de fato não se aplicam ao nosso projeto?
Por exemplo:

  • *.so
  • Flask stuff?
  • spyderproject
  • rope?
  • mypy?
  • celery?
  • etc

Eu adicionaria apenas o que for relevante ao projeto.
Adicionando a @LivSith na discussão.

@LivSith
Copy link
Collaborator

LivSith commented Mar 6, 2021

Sim, @Delatorrea seria bom o gitigonre refletir o nosso projeto no momento q ele está agora, não pensando nos arquivos q ele terá no futuro.

Para isso seria bom, se perguntar "qual arquivo eu tenho na pasta do projeto q não é para ir para o github?" - essa pergunta ajuda a criar um arquivo (gitignore) simples e q refletirá o estagio atual do projeto.

E uma duvida, vc está usando venv ou vagrant para rodar local o projeto? se estiver usando vagrant poderia incluir no README (na parte final do arquivo) como rodar o projeto com vagrant...

@fazedordecodigo
Copy link
Collaborator Author

fazedordecodigo commented Mar 6, 2021

Show. Vou retirar os excessos.
@LivSith to usando venv, nem conhecia esse vagrant. Hehehe. Vou pesquisar aqui.

.gitignore Outdated

# C extensions
*.so
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esse é necessário?

.gitignore Outdated
# Translations
*.mo
*.pot
Copy link
Collaborator

Choose a reason for hiding this comment

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

Precisamos desses dois?

.gitignore Outdated

# mkdocs documentation
/site
Copy link
Collaborator

Choose a reason for hiding this comment

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

Qual a ideia desse diretorio?

.gitignore Outdated
# Python cache files
**/**.pyc
# Vagrant
.vagrant/
Copy link
Collaborator

Choose a reason for hiding this comment

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

Alguém vai usar vagrant?

Copy link
Collaborator

@LivSith LivSith left a comment

Choose a reason for hiding this comment

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

@Delatorrea Responde as duvidas, mas é mais para entender o q está acontecendo no seu projeto, pq no tamanho q está o nosso projeto, se não me engano o gitignore seria de no máximo 5 linhas. [Isto não trava deploy]

@@ -63,6 +53,3 @@ venv/
ENV/
env.bak/
venv.bak/
Copy link
Collaborator

Choose a reason for hiding this comment

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

aqui nos Environments, como no README já estamos falando para criar uma maquina virtual chamada venv, não precisamos mais tratar todos os possiveis nomes de ambientes virtuais q seriam criados.

@@ -17,9 +14,6 @@ __pycache__/
*.py[cod]
*$py.class

# C extensions
*.so

# Distribution / packaging
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Delatorrea esses pacotes estão aparecendo na sua pastas server? ou em subpastas dentro da server? se estão em subpastas, qual o nome da pasta q possui eles?

@fazedordecodigo
Copy link
Collaborator Author

@Delatorrea Responde as duvidas, mas é mais para entender o q está acontecendo no seu projeto, pq no tamanho q está o nosso projeto, se não me engano o gitignore seria de no máximo 5 linhas. [Isto não trava deploy]

Tranquilo, na verdade eu estava em busca de um padrão de gitignore para travar todas as extensões e pastas desnecessárias... Não tenho nada em específico no meu projeto.

@fazedordecodigo fazedordecodigo merged commit 96e3e88 into master Mar 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants