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

Runtime changes common #4180

Merged
merged 40 commits into from
Jul 26, 2023
Merged

Runtime changes common #4180

merged 40 commits into from
Jul 26, 2023

Conversation

Eitu33
Copy link
Contributor

@Eitu33 Eitu33 commented Jun 30, 2023

No description provided.

massa-grpc/src/api.rs Outdated Show resolved Hide resolved
@@ -27,7 +27,7 @@ massa_final_state = { path = "../massa-final-state" }
massa_ledger_exports = { path = "../massa-ledger-exports", optional = true }
massa_module_cache = { path = "../massa-module-cache" }
massa_versioning = { path = "../massa-versioning" }
massa-sc-runtime = { git = "https://github.com/massalabs/massa-sc-runtime", branch = "main" }
massa-sc-runtime = { git = "https://github.com/massalabs/massa-sc-runtime", branch = "feature/Improve_ABI_types_in_wasmv1" }
Copy link
Contributor

@Leo-Besancon Leo-Besancon Jun 30, 2023

Choose a reason for hiding this comment

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

/!\ Do not merge until we:

  • Merge the massa-sc-runtime common branch into main
  • Update the Cargo.toml with a rev of the massa-sc-runtime commit / (or we target main instead?)

@Leo-Besancon Leo-Besancon added the blocked Issues that can't be done for now. label Jun 30, 2023
* Update interface impl for leo's changes

Depends on feature/Improve_ABI_types_in_wasmv1 branch for massa-sc-runtime and massa-proto-rs

Signed-off-by: Jean-François Morcillo <[email protected]>

Update target branch

Implemented the interface for my new ABIs

Added prefix to get_op_keys_wasmv1

Update target branch

Update in_downtime context value

Add some unit tests

Use helper function to get address

Added comments and deprecation notice

Target new runtime rev to have consistant proto-rs targets

Simplified a bit the prefix logic

Based on review comments

* Fix after rebase

---------

Co-authored-by: Jean-François Morcillo <[email protected]>
@AurelienFT AurelienFT changed the base branch from testnet_24 to testnet_25 July 4, 2023 08:31
bilboquet and others added 14 commits July 5, 2023 11:26
* fmt

* Update branch targeting

* Implement abis

* Update interface (clippy, implem some abis..)

* fmt

* Changed back branch targeting
Co-authored-by: Jean-François Morcillo <[email protected]>
* Implement unsafe_random_wasmv1

* Added doc comments and deprecation notes
Signed-off-by: Jean-François Morcillo <[email protected]>
Signed-off-by: Jean-François Morcillo <[email protected]>
Signed-off-by: Jean-François Morcillo <[email protected]>
@Leo-Besancon Leo-Besancon removed the blocked Issues that can't be done for now. label Jul 26, 2023
@Eitu33 Eitu33 merged commit 494b095 into testnet_25 Jul 26, 2023
4 checks passed
@AurelienFT AurelienFT deleted the runtime-changes-common branch October 23, 2023 08:48
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.

4 participants