-
Notifications
You must be signed in to change notification settings - Fork 5
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
chore/bump oa 0.3.1 #29
Conversation
1424bdc
to
f46cdfd
Compare
python/mypy#13627 |
@@ -72,7 +72,7 @@ jobs: | |||
strategy: | |||
matrix: | |||
os: [ ubuntu-latest ] | |||
python-version: [ "3.10" ] | |||
python-version: [ "3.10.6" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why so explicit? I think we shouldn't do that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it has to do with this.
It's not architecture dependent, it's only about the specific version of Python you have in your image. If you have Python 3.10.7 you'll see the error, if you have Python 3.10.6 you won't. In all cases, upgrading to mypy 0.981 will fix.
I can reproduce the problem locally when running tox -e mypy.
@@ -21,8 +21,8 @@ | |||
"skill/valory/transaction_settlement_abci/0.1.0": "bafybeia2gs36s2mz32yaznhlidiolhw5swkumiej4vgwywphppjvurldie", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should also update this to the matching hashes here: https://github.com/valory-xyz/open-autonomy/blob/v0.3.1/packages/packages.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR bumps the repo to
open-autonomy@v0.3.1
andopen-aea@1.20.0
.