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

Refactor - Move Executor in program-runtime crate #28782

Merged

Conversation

Lichtso
Copy link
Contributor

@Lichtso Lichtso commented Nov 10, 2022

Problem

For the new loader built-in program it would be useful to share these.

Summary of Changes

  • Moves CreateMetrics into the program-runtime crate.
  • Moves the Executor trait into executor.rs
  • Removes the first_instruction_account parameter from Executor::execute().

@Lichtso Lichtso force-pushed the refactor/move_executor_in_program_runtime branch from 02a53af to a14cba4 Compare November 10, 2022 15:51
@Lichtso Lichtso force-pushed the refactor/move_executor_in_program_runtime branch from a14cba4 to 2950818 Compare November 10, 2022 16:02
@Lichtso Lichtso merged commit 4142f42 into solana-labs:master Nov 10, 2022
@Lichtso Lichtso deleted the refactor/move_executor_in_program_runtime branch November 10, 2022 20:11
xiangzhu70 pushed a commit to xiangzhu70/solana that referenced this pull request Nov 16, 2022
* Moves CreateMetrics into the program-runtime crate.

* Moves the Executor trait into executor.rs

* Removes the first_instruction_account parameter from Executor::execute().
gnapoli23 pushed a commit to gnapoli23/solana that referenced this pull request Dec 16, 2022
* Moves CreateMetrics into the program-runtime crate.

* Moves the Executor trait into executor.rs

* Removes the first_instruction_account parameter from Executor::execute().
nickfrosty pushed a commit to nickfrosty/solana that referenced this pull request Jan 4, 2023
* Moves CreateMetrics into the program-runtime crate.

* Moves the Executor trait into executor.rs

* Removes the first_instruction_account parameter from Executor::execute().
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.

1 participant