-
Notifications
You must be signed in to change notification settings - Fork 95
smartcontract: add support NEP-25 #4036
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
base: master
Are you sure you want to change the base?
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #4036 +/- ##
==========================================
+ Coverage 72.48% 72.76% +0.28%
==========================================
Files 364 365 +1
Lines 56750 57147 +397
==========================================
+ Hits 41135 41583 +448
+ Misses 13880 13821 -59
- Partials 1735 1743 +8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
- semantic checks
- stack item serialization
See neo-project/neo#4043 also
| Overloads map[string]string `yaml:"overloads,omitempty"` | ||
| NamedTypes map[string]binding.ExtendedType `yaml:"namedtypes,omitempty"` | ||
| Overloads map[string]string `yaml:"overloads,omitempty"` | ||
| NamedTypes map[string]manifest.ExtendedType `yaml:"namedtypes,omitempty"` |
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.
Likely it's no longer needed here, manifest has it all.
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.
Do I need to write an UnmarshalYAML method for ProjectConfig or mark the NamedTypes field as deprecated?
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.
Just drop them and then let's see how it goes.
|
|
||
| import ( | ||
| "encoding/json" | ||
| "gopkg.in/yaml.v3" |
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.
Likely we no longer need yaml here.
1bf2759 to
0457d1a
Compare
Close #3995. Signed-off-by: Tural Devrishev <[email protected]>
0457d1a to
a8966da
Compare
Close #3995.