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

Release 0.5.0 #1461

Merged
merged 400 commits into from
Jul 6, 2020
Merged

Release 0.5.0 #1461

merged 400 commits into from
Jul 6, 2020

Conversation

DavidMinarsch
Copy link
Contributor

@DavidMinarsch DavidMinarsch commented Jul 4, 2020

Release summary

Version number: v0.5.0

Release details

This release brings:

  • Refactors all connections to be fully async friendly
  • Adds almost complete test coverage on connections
  • Adds complete test coverage for cli and cli gui
  • Fixes cli gui functionality and removes oef node dependency
  • Refactors p2p go code and increases test coverage
  • Refactors protocol generator for higher code reusability
  • Adds option for skills to depend on other skills
  • Adds abstract skills option
  • Adds ledger connections to execute ledger related queries and transactions, removes ledger apis from skill context
  • Adds contracts registry and removes them from skill context
  • Rewrites all skills to be fully message based
  • Replaces internal messages with protocols (signing and state update)
  • Multiple refactoring to improve pylint adherence
  • Multiple docs updates
  • Multiple test stability fixes

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING doc
  • I am making a pull request against the master branch (left side), from develop
  • Lint and unit tests pass locally
  • I have checked the fingerprint hashes are correct by running (scripts/generate_ipfs_hashes.py)
  • I have regenerated the latest API docs
  • I built the documentation and updated it with the latest changes
  • I have added an item in HISTORY.md for this release
  • I bumped the version number in the aea/__version__.py file.
  • I bumped the version number in the docs/version.md file
  • I bumped the version number in every Docker image of the repo and published it. Also, I built and published them with tag latest
    (check the READMEs of aea-develop
    and aea-deploy)
  • I have checked that the documentation about the aea cli tool works
  • I have pushed the latest packages to the registry.

Further comments

lrahmani and others added 30 commits June 23, 2020 15:43
Test coverage gaps for interact and scaffold filled AEA-629
raise error at build time if more than one model with the same name
filter out those whose module path starts with 'packages.'
update duplicate skill model classes check
[AEA-650] [AEA-669] [AEA-671] Refactor Libp2p DHT peer - Add golang to ci
@codecov-commenter
Copy link

codecov-commenter commented Jul 4, 2020

Codecov Report

Merging #1461 into master will decrease coverage by 0.92%.
The diff coverage is 84.84%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1461      +/-   ##
==========================================
- Coverage   90.49%   89.57%   -0.93%     
==========================================
  Files         206      242      +36     
  Lines       14945    17183    +2238     
==========================================
+ Hits        13525    15391    +1866     
- Misses       1420     1792     +372     
Flag Coverage Δ
#unittests 89.57% <84.84%> (-0.93%) ⬇️
Impacted Files Coverage Δ
aea/connections/base.py 100.00% <ø> (+5.10%) ⬆️
aea/connections/scaffold/connection.py 100.00% <ø> (+6.66%) ⬆️
aea/helpers/search/generic.py 100.00% <ø> (ø)
aea/helpers/win32.py 100.00% <ø> (+9.09%) ⬆️
packages/fetchai/connections/gym/connection.py 95.74% <ø> (+0.04%) ⬆️
...ages/fetchai/connections/http_client/connection.py 100.00% <ø> (+13.75%) ⬆️
...ages/fetchai/connections/http_server/connection.py 100.00% <ø> (+6.03%) ⬆️
packages/fetchai/connections/ledger/__init__.py 100.00% <ø> (ø)
packages/fetchai/connections/ledger/base.py 94.91% <ø> (ø)
packages/fetchai/connections/ledger/connection.py 90.54% <ø> (ø)
... and 178 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ec16af5...6b8b84d. Read the comment docs.

DavidMinarsch and others added 25 commits July 4, 2020 16:25
It wasn't needed in the current state of the implementation.
…r-connection

Fix check of receiving task list in Ledger API connection
fix requests version to avoid clashes
Copy link
Contributor

@marcofavorito marcofavorito left a comment

Choose a reason for hiding this comment

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

LGTM

@DavidMinarsch DavidMinarsch merged commit cc707c2 into master Jul 6, 2020
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.

7 participants