Skip to content

🐛 Fix status process in tests #361

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

Merged
merged 5 commits into from
Jul 19, 2022
Merged

Conversation

hillairet
Copy link
Collaborator

There was an issue in the way the success of the tests is reported. For example if mypy was missing it would say:

######### CHECK TYPING: OK
mypy command not found

Not great.
Now it will say "FAILED"

Also I cleaned up the dependencies in the pyproject.toml and added the poetry.lock cause we want consistent testing and that's how it'll be in apps.

Finally I removed a warning in pytest by adding an entry in the pyproject.toml.

@hillairet hillairet added enhancement New feature or request docker Pull requests that update Docker code labels Jul 18, 2022
@hillairet hillairet self-assigned this Jul 18, 2022
@hillairet hillairet merged commit 32c201e into main Jul 19, 2022
@hillairet hillairet deleted the fix/status-process-in-tests branch July 19, 2022 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Pull requests that update Docker code enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants