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

fix: Fix API dependencies #748

Merged
merged 2 commits into from
Aug 14, 2024
Merged

fix: Fix API dependencies #748

merged 2 commits into from
Aug 14, 2024

Conversation

paulespinosa
Copy link
Member

@paulespinosa paulespinosa commented Aug 14, 2024

This commit fixes a syntax error in pyproject.toml and also fixes erroneous dependency updates caused by dependabot.

closes #746

What changes did you make?

Fixed a syntax error in pyproject.toml and also fixed erroneous dependency updates caused by dependabot.

Rationale behind the changes?

Dependabot caused a version conflict amongst connexion and werkzerg

Testing done for these changes

pip-sync to update virtual environment dependencies
Run tox to test API.

This commit fixes a syntax error in `pyproject.toml` and also fixes erroneous
dependency updates caused by dependabot.

closes #746
Originally the `pyjwt` library was added directly to `requirements.txt`. When
adding dependencies, add them to `pyproject.toml` and the run the commands
documented in that file to regenerate `requirements.txt` and `requirements-dev.txt`.
@paulespinosa paulespinosa added the bug Something isn't working label Aug 14, 2024
@paulespinosa paulespinosa self-assigned this Aug 14, 2024
Copy link
Member

@johnwroge johnwroge left a comment

Choose a reason for hiding this comment

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

Thanks Paul. This looks good to me.

@johnwroge johnwroge merged commit 7572f48 into main Aug 14, 2024
3 of 6 checks passed
@paulespinosa paulespinosa deleted the 746-bug-api-reqs branch August 14, 2024 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: API requirements.txt dependencies are wrong
2 participants