Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Force futures version to be at least 0.18 #585

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.

Sign up for GitHub

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

Jump to bottom
Merged
amotl merged 1 commit into from
Sep 15, 2022
Merged

Force futures version to be at least 0.18 #585

amotl merged 1 commit into from
Sep 15, 2022

Conversation

@arieroos
Copy link
Contributor

@arieroos arieroos commented Sep 15, 2022

I had some issues installing mqttwarn in an existing venv. This fixes #584 which is my issue.

I tested my change on my dev machine with these commands:

 pyenv install 3.10.4
 pyenv shell 3.10.4
 git clone [email protected]:arieroos/mqttwarn.git
 cd mqttwarn
 pip --version
 > pip 22.0.4 from /Users/arieroos/.pyenv/versions/3.10.4/lib/python3.10/site-packages/pip (python 3.10)
 pip install future==0.17.1
 pip install .
 mqttwarn make-config > mqttwarn.ini
 cat mqttwarn.ini

All commands succeeded

@amotl amotl changed the title Force future version to be at leat 0.18 Force futures version to be at least 0.18 Sep 15, 2022
amotl
amotl approved these changes Sep 15, 2022
View reviewed changes
Copy link
Member

@amotl amotl left a comment

Choose a reason for hiding this comment

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

Thanks a stack!

@amotl amotl merged commit 4372204 into mqtt-tools:main Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@amotl amotl amotl approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Python 3.10 support

2 participants

@arieroos @amotl