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

Atualização do README.md #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JacksonOsvaldo
Copy link

Olá. Sugeri algumas mudanças e inserções no README.md do exemplo de implantação no Flask. Espero que esteja dentro do esperado. Qualquer coisa, estou a disposição para esclarecimentos.

Obs.: Considerar o último commit (Atualizando README.md - eb09d38).

:)

==========

```bash
virtualenv env
pip install -U pip
pip install virtualenv
Copy link
Contributor

Choose a reason for hiding this comment

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

Eu acho que você poderia utilizar o pipenv ou poetry ao invés de criar o virtualenv na mão, apesar destas libs utilizar por debaixo dos panos o virtualenv, ganhamos algumas features como versionamento, lock file e etc...

Com o terminal aberto e setado na pasta _python-pagseguro/examples/flask_, execute o seguinte comando:

```bash
env FLASK_APP=run.py flask run
Copy link
Contributor

Choose a reason for hiding this comment

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

FLASK_APP é uma variável de ambiente, utilize o export para exporta-la. Ou sugiro também add uma referência para o SET do CMD Windows.

Referência: https://flask.palletsprojects.com/en/1.0.x/cli/


```bash
chmod +x tests.py
Copy link
Contributor

Choose a reason for hiding this comment

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

Eu ainda estou em dúvida sobre isso, mas eu acredito que não precisa da permissão para rodar os testes. Caso sinta dificuldade de utilizar o Makefile rode manualmente assim: pytest <test_file.py>.

O que acha @mazulo e @rochacbruno ?

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