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: bot and web_api build #731

Closed
wants to merge 4 commits into from
Closed

fix: bot and web_api build #731

wants to merge 4 commits into from

Conversation

chdsbd
Copy link
Owner

@chdsbd chdsbd commented Sep 22, 2021

The Poetry install for purest was failing because it's dependency, pluggy was failing to install because it's dependency, zipp was failing to install.

Bumping the pytest version fixes the install, but I also needed to upgrade pytest-asyncio to fix a deprecation error.

@chdsbd chdsbd changed the title test build fix(bot): build Sep 22, 2021
@chdsbd chdsbd requested a review from sbdchd September 22, 2021 02:06
Copy link
Collaborator

@sbdchd sbdchd left a comment

Choose a reason for hiding this comment

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

I think the API pyproject file also needs to be updated

@chdsbd chdsbd changed the title fix(bot): build fix: bot and web_api build Sep 22, 2021
@sbdchd
Copy link
Collaborator

sbdchd commented Sep 22, 2021

Looks like we might need to upgrade our poetry version to avoid that runtime bug: python-poetry/poetry#4409 (comment)

@chdsbd
Copy link
Owner Author

chdsbd commented Sep 23, 2021

#732 should resolve this

@chdsbd chdsbd closed this Sep 23, 2021
@chdsbd chdsbd deleted the chris/test-build branch September 23, 2021 02:05
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.

None yet

2 participants