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(avm): address bytecode hashing comments #9436

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

IlyasRidhuan
Copy link
Contributor

@IlyasRidhuan IlyasRidhuan commented Oct 25, 2024

Fixing up some earlier comments in PRs

@IlyasRidhuan IlyasRidhuan force-pushed the ir/09-28-feat_trace_class_id_and_instance branch from 5ff45f6 to 3395ccf Compare October 25, 2024 16:14
@IlyasRidhuan IlyasRidhuan force-pushed the ir/10-25-fix_avm_address_bytecode_hashing_comments branch from 2608730 to 1385a08 Compare October 25, 2024 16:15
@dbanks12
Copy link
Contributor

I'd change or comment the line mentioned here
#8897 (comment)

@IlyasRidhuan IlyasRidhuan force-pushed the ir/09-28-feat_trace_class_id_and_instance branch 2 times, most recently from 26249b2 to b6c741e Compare October 26, 2024 17:38
@IlyasRidhuan IlyasRidhuan force-pushed the ir/10-25-fix_avm_address_bytecode_hashing_comments branch from 1385a08 to 5ce56b4 Compare October 26, 2024 17:39
Base automatically changed from ir/09-28-feat_trace_class_id_and_instance to master October 26, 2024 22:26
@IlyasRidhuan IlyasRidhuan force-pushed the ir/10-25-fix_avm_address_bytecode_hashing_comments branch from 5ce56b4 to 54c1e3d Compare October 27, 2024 13:32
@IlyasRidhuan IlyasRidhuan marked this pull request as ready for review October 27, 2024 13:37
@dbanks12 dbanks12 merged commit a85f92a into master Oct 27, 2024
100 of 101 checks passed
@dbanks12 dbanks12 deleted the ir/10-25-fix_avm_address_bytecode_hashing_comments branch October 27, 2024 15:02
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