Skip to content

Add support for scarb execute#222

Merged
THenry14 merged 5 commits intomainfrom
szymczyk/192-execute
Oct 20, 2025
Merged

Add support for scarb execute#222
THenry14 merged 5 commits intomainfrom
szymczyk/192-execute

Conversation

@THenry14
Copy link
Copy Markdown
Member

@THenry14 THenry14 commented Sep 28, 2025 •
edited
Loading

Closes #192
Connected to software-mansion/scarb#2644

Introduced changes

  • Adds support for scarb execute

Checklist

  • Linked relevant issue
  • Updated relevant documentation (README.md)
  • Added relevant tests
  • Performed self-review of the code
  • Added changes to CHANGELOG.md

@THenry14 THenry14 force-pushed the szymczyk/192-execute branch 4 times, most recently from cbb92bf to a027839 Compare October 3, 2025 21:14
THenry14
THenry14 commented Oct 3, 2025
View reviewed changes
Comment thread README.md Outdated
@THenry14 THenry14 marked this pull request as ready for review October 3, 2025 21:20
@THenry14 THenry14 requested a review from ksew1 October 3, 2025 21:20
@THenry14 THenry14 force-pushed the szymczyk/192-execute branch from a027839 to 15ef8ea Compare October 6, 2025 21:21
@THenry14 THenry14 force-pushed the szymczyk/192-execute branch from 15ef8ea to 5f3d4d6 Compare October 6, 2025 21:33
ksew1
ksew1 reviewed Oct 13, 2025
View reviewed changes
Comment thread crates/cairo-profiler/src/cli/build_profile.rs Outdated
Comment thread crates/cairo-profiler/src/trace_reader/function_name.rs Outdated
Comment thread crates/cairo-profiler/src/trace_reader/function_name.rs Outdated
Comment thread crates/cairo-profiler/src/trace_reader/function_name.rs
Comment thread crates/cairo-profiler/src/trace_reader/function_trace_builder.rs Outdated
Comment thread crates/cairo-profiler/src/trace_reader/function_trace_builder.rs Outdated
@THenry14 THenry14 force-pushed the szymczyk/192-execute branch from 85384dc to aada5dc Compare October 14, 2025 07:48
@THenry14 THenry14 requested a review from ksew1 October 14, 2025 07:49
ksew1
ksew1 reviewed Oct 14, 2025
View reviewed changes
Comment thread crates/cairo-profiler/src/trace_reader/function_trace_builder.rs Outdated
Comment thread crates/cairo-profiler/src/trace_reader/function_name.rs Outdated
@THenry14 THenry14 requested a review from ksew1 October 14, 2025 12:33
@THenry14 THenry14 force-pushed the szymczyk/192-execute branch from fe516c7 to ff3cdf9 Compare October 20, 2025 08:03
@THenry14 THenry14 enabled auto-merge October 20, 2025 08:04
@THenry14 THenry14 added this pull request to the merge queue Oct 20, 2025
Merged via the queue into main with commit 0e031e4 Oct 20, 2025
4 checks passed
@THenry14 THenry14 deleted the szymczyk/192-execute branch October 20, 2025 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ksew1 ksew1 ksew1 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Integration with scarb execute

2 participants

@THenry14 @ksew1