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

examples: update examples #1296

Merged
merged 5 commits into from
Aug 11, 2020
Merged

examples: update examples #1296

merged 5 commits into from
Aug 11, 2020

Conversation

AnnaShaleva
Copy link
Member

Closes #1234

@codecov
Copy link

codecov bot commented Aug 10, 2020

Codecov Report

Merging #1296 into master will increase coverage by 2.47%.
The diff coverage is 21.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1296      +/-   ##
==========================================
+ Coverage   65.72%   68.20%   +2.47%     
==========================================
  Files         217      217              
  Lines       18564    18570       +6     
==========================================
+ Hits        12201    12665     +464     
+ Misses       5703     5220     -483     
- Partials      660      685      +25     
Impacted Files Coverage Δ
cli/smartcontract/smart_contract.go 0.00% <0.00%> (ø)
pkg/compiler/compiler.go 53.40% <0.00%> (ø)
pkg/compiler/debug.go 81.10% <100.00%> (+1.46%) ⬆️
pkg/compiler/codegen.go 89.63% <0.00%> (-0.20%) ⬇️
pkg/core/transaction/transaction.go 77.77% <0.00%> (+1.23%) ⬆️
pkg/rpc/client/nep5.go 3.60% <0.00%> (+3.60%) ⬆️
pkg/rpc/request/types.go 82.60% <0.00%> (+26.08%) ⬆️
pkg/rpc/client/policy.go 55.26% <0.00%> (+55.26%) ⬆️
pkg/rpc/client/client.go 56.62% <0.00%> (+56.62%) ⬆️
pkg/rpc/client/rpc.go 77.51% <0.00%> (+77.51%) ⬆️
... and 1 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 d8db85e...e2b22bc. Read the comment docs.

@AnnaShaleva AnnaShaleva force-pushed the smartcontract/examples branch 4 times, most recently from f4099e1 to f8caf4e Compare August 10, 2020 15:36
examples/engine/engine.yml Outdated Show resolved Hide resolved
examples/token/nep5/nep5.go Show resolved Hide resolved
examples/engine/engine.yml Show resolved Hide resolved
examples/token-sale/token_sale.go Outdated Show resolved Hide resolved
@AnnaShaleva AnnaShaleva force-pushed the smartcontract/examples branch 3 times, most recently from d7af951 to 1e5150c Compare August 11, 2020 08:45
We currently can't process events in codegen, so we have to provide
them via .yml config file. Do not delete the rest of the code connected
with conversion of MethodDebugInfo.Event into manifest.Event as we have
issue #1038.
examples/token/nep5/nep5.go Outdated Show resolved Hide resolved
We don't have totalSupply in contract storage because we did't put it
there. Fixed.
@roman-khimov roman-khimov merged commit c3f7a41 into master Aug 11, 2020
@roman-khimov roman-khimov deleted the smartcontract/examples branch August 11, 2020 16:09
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.

Update smartcontract examples
2 participants