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
[refactor] Validate item names/ids without allocating Vec #3003
Conversation
Signed-off-by: Wenfeng Sui <suiwenfeng@qq.com>
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.
Good job!
Here few minor things you should do:
- We use standardize format for commit names, so you commit message should look like:
[refactor] #2675: Validate item names/ids without allocating Vec
. - Also we use
rustfmt
in our workflow , you can run it by following command:cargo +nightly fmt --all
assert!(account_empty.is_err()); | ||
|
||
let account_invalid: Result<AccountId, _> = "testhello".parse(); | ||
assert!(account_invalid.is_err()); |
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.
Wight be redundant but we can also test for string "@"
.
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 reopened a pr for some reason, you have to check this at #3004 now.
Codecov Report
@@ Coverage Diff @@
## iroha2-dev #3003 +/- ##
==============================================
- Coverage 67.61% 62.47% -5.14%
==============================================
Files 140 169 +29
Lines 26173 30948 +4775
==============================================
+ Hits 17696 19334 +1638
- Misses 8477 11614 +3137
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Signed-off-by: Shanin Roman <shanin1000@yandex.ru>
Signed-off-by: Wenfeng Sui suiwenfeng@qq.com
Signed-off-by: Wenfeng Sui suiwenfeng@qq.com
Description of the Change
Issue
Benefits
Possible Drawbacks
Usage Examples or Tests [optional]
Alternate Designs [optional]