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

querido-diario-data-processing para Windows #39

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Luisa-Coelho
Copy link

O MakeFile tem alguns comandos diferentes incluindo a remoção de "cp":
substituir cp --no-clobber contrib/sample.env envvars || true POR copy /y contrib\sample.env envvars

O comando "wait-for" substituído por:
run-command=(podman run --rm -ti --volume $(PWD):/mnt/code:rw
--pod $(POD_NAME)
--env PYTHONPATH=/mnt/code
--env POSTGRES_PASSWORD=$(POSTGRES_PASSWORD)
--env POSTGRES_USER=$(POSTGRES_USER)
--env POSTGRES_DB=$(POSTGRES_DB)
--env POSTGRES_HOST=$(POSTGRES_HOST)
--env POSTGRES_PORT=$(POSTGRES_PORT)
$(IMAGE_NAMESPACE)/$(IMAGE_NAME):$(IMAGE_TAG) $1)

Além disso, foi necessário deixar requirements.txt na pasta scripts. O comando COPY no DockerFile não funcionou para Windows.

Não testei todas as funcionalidades do data-processing, mas não dá para fazer um merge direto pois existem essas mudanças no Makefile.

@ogecece ogecece added the documentação Improvements or additions to documentation label Sep 20, 2023
Copy link
Collaborator

@jvanz jvanz left a comment

Choose a reason for hiding this comment

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

@Luisa-Coelho, antes de mais nada, desculpe pela demora para revisar.

Por favor, remova da PR os arquivos da pasta .venv. Como são arquivos do virtual env, acredito que eles não devem ser adicionados no repositório. Já que cada usuário vai ter seu venv local. Depois que vc remove, já adiciona esse diretório (.venv) no .gitignore. Assim não vamos ter esse problema no futuro.

@Luisa-Coelho
Copy link
Author

Oi, @jvanz ! Vou atualizar, mas na verdade essa é uma modificação que comentaram não ser muito bem-vinda, já que o QD roda primordialmente no Linux.

Luisa-Coelho and others added 5 commits April 11, 2024 10:28
Signed-off-by: Luísa F. Coelho <87907716+Luisa-Coelho@users.noreply.github.com>
Signed-off-by: Luísa F. Coelho <87907716+Luisa-Coelho@users.noreply.github.com>
Signed-off-by: Luísa F. Coelho <87907716+Luisa-Coelho@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentação Improvements or additions to documentation
Projects
Development

Successfully merging this pull request may close these issues.

3 participants