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

[Feature]Add string module #3626

Merged
merged 9 commits into from
Aug 16, 2022
Merged

[Feature]Add string module #3626

merged 9 commits into from
Aug 16, 2022

Conversation

WGB5445
Copy link
Contributor

@WGB5445 WGB5445 commented Aug 7, 2022

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

@WGB5445 WGB5445 requested a review from nkysg August 7, 2022 08:43
@codecov
Copy link

codecov bot commented Aug 7, 2022

Codecov Report

❗ No coverage uploaded for pull request base (update_move@084a3e1). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             update_move    #3626   +/-   ##
==============================================
  Coverage               ?   28.98%           
==============================================
  Files                  ?      594           
  Lines                  ?    50606           
  Branches               ?    23865           
==============================================
  Hits                   ?    14662           
  Misses                 ?    21395           
  Partials               ?    14549           
Flag Coverage Δ
unittests 28.98% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 084a3e1...7ad9855. Read the comment docs.

@WGB5445 WGB5445 marked this pull request as ready for review August 9, 2022 08:24
@nkysg nkysg merged commit 6a2d5ac into starcoinorg:update_move Aug 16, 2022
nanne007 added a commit to nanne007/starcoin that referenced this pull request Oct 10, 2022
* [compiler] update move
[compiler] Upgrade Move and fix compiler error
[clippy] Fix clippy error

* Merge branch 'master' into update_move

* gen new stdlib

* fix warning: add phantom for merkle distributor

* update move version
fix merkle tree test

* fix Identifier merge

* Update module_upgrade_test.rs

* update script
update move

* check commit

* 1.add table native functions
2.gen new stdlib
3.add table native function gas cost in vmconfig

* 1.add StateKey,MoveExt, SessionId,MoveResolveExt
2.add session finish_with_extension
3.change WriteSet , TransactionOutput, StateKeyView

* add contract_api.json

* 1.add halley genesis

* gen new stdlib

* Merge branch 'master' into update_move

* fix lock

* update move repo

* add move table extension flush

* add statedb support move table extension with commit and flush

* remove WriteAccessPathSet and refactor code

* remove unused code

* add halley genesis

* add halley genesis

* remove file

* fix bug

* add halley genesis

* add table item test code

* fix  warnings

* update move repo

* fix fmt

* reset proxima

* add proxima genesis_config.json

* branch update_move add docker build

* add table_info storage

* update starcoin-framework repo

* fix clippy warnings

* reset proxima genesis

* update docker build

* update starcoin_framework

* gen new stdlib, reset proxima genesis

* gen new stdlib, reset proxima genesis

* gen new stdlib, reset proxima genesis

* gen new stdlib, reset proxima genesis

* add TableItemProof

* add TableItemProof

* fmt Cargo.toml

* add TableItemProof unit_test

* change StateKeyView serde Field attributes

* fix pre_lib

* [Feature]Add string module (starcoinorg#3626)

* add string

* fix string

* add string in stdlib

* fix stdlib

* new stdlib

* Fix String module native

* Fix cargo move

* update move commit

* add manual dispatch to workflow build and test

* add pr build_test

* fix execute script (starcoinorg#3705)

* add config

* fix run script

* fix mpm package table test (starcoinorg#3715)

* [dev branch]table extension compatible dry run (starcoinorg#3706)

* change dry_out return value

* add DryRunOutputViewV2 for StateKey

* add dry_run_table_item, dry_run_state_key

* remove dry_run_table_item, dry_run_state_key

* reset proxima

* reset proxima

* close starcoinorg#3732

* mpm: remove tmp code

Co-authored-by: LemonHX <[email protected]>
Co-authored-by: nk_ysg <[email protected]>
Co-authored-by: WGB5445 <[email protected]>
nkysg added a commit that referenced this pull request Feb 10, 2023
* [compiler] update move
[compiler] Upgrade Move and fix compiler error
[clippy] Fix clippy error

* Merge branch 'master' into update_move

* gen new stdlib

* fix warning: add phantom for merkle distributor

* update move version
fix merkle tree test

* fix Identifier merge

* Update module_upgrade_test.rs

* update script
update move

* check commit

* 1.add table native functions
2.gen new stdlib
3.add table native function gas cost in vmconfig

* 1.add StateKey,MoveExt, SessionId,MoveResolveExt
2.add session finish_with_extension
3.change WriteSet , TransactionOutput, StateKeyView

* add contract_api.json

* 1.add halley genesis

* gen new stdlib

* Merge branch 'master' into update_move

* add move table extension flush

* add statedb support move table extension with commit and flush

* add table item test code

* fix  warnings

* update move repo

* fix fmt

* reset proxima

* add proxima genesis_config.json

* branch update_move add docker build

* add table_info storage

* update starcoin-framework repo

* fix clippy warnings

* gen new stdlib, reset proxima genesis

* add TableItemProof

* add TableItemProof

* fmt Cargo.toml

* add TableItemProof unit_test

* change StateKeyView serde Field attributes

* [Feature]Add string module (#3626)

* add string

* fix string

* add string in stdlib

* new stdlib

* Fix String module native

* Fix cargo move

* add pr build_test

* fix execute script (#3705)

* add config

* fix run script

* fix mpm package table test (#3715)

* [dev branch]table extension compatible dry run (#3706)

* change dry_out return value

* reset proxima

* close #3732

* mpm: remove tmp code

* [dev]Update docker_build.yml docker/setup-buildx-action to 2,  Dockerfile rustup version to 1.25.1 (#3771)

* Update docker_build.yml

* Update Dockerfile

* [dev] remove starcoin-x package,update script and config (#3772)

* [dev] remove starcoin-x package

* remove in toml

* update mold flag

* remove starcoin-x alias

* add back test scripts

* fix powershell

* use BAD_TRANSACTION_FEE_CURRENCY for test

gas_schedule: change max gas unit of test env to 400_000_000

[dev branch] move update use new  gas meter imp (#3795)

* use GasParameter replace costTable

* compat GasCost and GasConstants

* ErrorContext change to ErrorDescription

* update GasConstants member function

* ErrorContext change to ErrorDescription

* update move TableHandle def

* 1.refactor gas-algebra-ext
2.add gas_params for starcoin_vm

* 1.add move stdlib nursery GasParam

* update starcoin_natives gas_params

* update VmConfig to gas_params

* add vmconfig to gas_schedule test

* add gas meter calc function

* update starcoin_vm get gas_params

* temp remove table_info column

* update starcoin-framework

* add test stdlib precompiled

* update move_stdlib params

* Package transaction publish bundle logic

* Update move dep for fixing transactional test harness

* comment StarcoinFramework checkpoint

* compatible legacy AccountDataCache

* move bytecode v6

* update starcoin-framework

Co-authored-by: jolestar <[email protected]>
Co-authored-by: fikgol <[email protected]>
Co-authored-by: WGB5445 <[email protected]>

* abi support u16, u32, u256

* update rpc schema

---------

Co-authored-by: LemonHX <[email protected]>
Co-authored-by: WGB5445 <[email protected]>
Co-authored-by: caojiafeng <[email protected]>
Co-authored-by: jiangying <[email protected]>
Co-authored-by: jolestar <[email protected]>
Co-authored-by: fikgol <[email protected]>
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.

2 participants