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

Install aiomysql from git and re-enable warnings #493

Merged
merged 3 commits into from
Sep 24, 2019

Conversation

Askaholic
Copy link
Collaborator

I submitted a PR to aiomysql to fix the deprecation warnings aio-libs/aiomysql#403 which was merged, but hasn't been deployed as apparently the people with the ability to do so have disappeared into the void. So instead, we can just install from git to get the latest version

@norraxx
Copy link
Collaborator

norraxx commented Sep 8, 2019

What do you miss in aiomysql, that is needed for you?

@norraxx
Copy link
Collaborator

norraxx commented Sep 8, 2019

Do you know, that you can uninstall things that aren't used in production? For example git?

@Askaholic
Copy link
Collaborator Author

Look at the PR I linked. It generates hundreds of warnings from inside aiomysql that make it very hard to find warnings that actually are caused by our code

@norraxx
Copy link
Collaborator

norraxx commented Sep 9, 2019

Lol! Np :-)

@Askaholic Askaholic merged commit 524e538 into FAForever:develop Sep 24, 2019
@Askaholic Askaholic deleted the install-aiomysql-from-git branch September 24, 2019 06:30
Brutus5000 pushed a commit that referenced this pull request Jan 27, 2020
* Install aiomysql from git and re-enable warnings

* Add git to dockerfile dependencies

* Update dependencies to latest versions
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