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

Fix sequencer number bug #1601

Merged
merged 31 commits into from
Apr 27, 2022
Merged

Fix sequencer number bug #1601

merged 31 commits into from
Apr 27, 2022

Conversation

asonnino
Copy link
Contributor

@asonnino asonnino added move Priority: High Very important task, not blocking but potentially delaying milestones or limiting our offering Status: In Progress Issue is actively being worked on sui-node Type: Bug Something isn't working labels Apr 26, 2022
@asonnino asonnino added this to the DevNet milestone Apr 26, 2022
@asonnino asonnino changed the title Fix bug https://github.com/MystenLabs/sui/issues/1600 Fix sequencer number bug Apr 26, 2022
@asonnino asonnino linked an issue Apr 26, 2022 that may be closed by this pull request
@asonnino asonnino marked this pull request as draft April 26, 2022 22:12
@asonnino asonnino marked this pull request as ready for review April 26, 2022 23:45
@asonnino asonnino requested review from gdanezis and LefKok April 26, 2022 23:45
@asonnino asonnino enabled auto-merge (squash) April 26, 2022 23:59
@asonnino asonnino self-assigned this Apr 27, 2022
@asonnino asonnino added Status: Needs Review PR is ready for review and removed Status: In Progress Issue is actively being worked on labels Apr 27, 2022
pub fn publish_move_package_transaction(gas_object: Object) -> Transaction {
let build_config = BuildConfig::default();
let mut path = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
path.push("../sui_programmability/examples/basics");
Copy link
Contributor

Choose a reason for hiding this comment

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

Might want to make this an argument to make this function more generic.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point #974

@asonnino asonnino merged commit 9174343 into main Apr 27, 2022
@asonnino asonnino deleted the counter-test branch April 27, 2022 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
move Priority: High Very important task, not blocking but potentially delaying milestones or limiting our offering Status: Needs Review PR is ready for review sui-node Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shared object index initialisation
2 participants