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.

Sign up for GitHub

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

Jump to bottom

Forte Rules Engine Gas Reports #484

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Forte Rules Engine Gas Reports #484

wants to merge 3 commits into from

Conversation

TJ-Everett
Copy link
Contributor

Updated GasReports.t.sol to no longer be used
Added GasReportV2 for ERC20 transfer focused gas tests

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 16, 2025 •
edited
Loading

Slither report

THIS CHECKLIST IS NOT COMPLETE. Use --show-ignored-findings to show all the results.
Summary

@github-actions GitHub Actions
Copy link

LCOV of commit f8c7b64 during Code coverage evaluation #294

Summary coverage rate:
  lines......: 81.6% (1836 of 2251 lines)
  functions..: 84.0% (441 of 525 functions)
  branches...: 66.6% (395 of 593 branches)

Files changed coverage rate: n/a

@TJ-Everett TJ-Everett requested a review from a team as a code owner January 17, 2025 17:14
@github-actions GitHub Actions
Copy link

LCOV of commit f825cec during Code coverage evaluation #297

Summary coverage rate:
  lines......: 81.6% (1836 of 2251 lines)
  functions..: 84.0% (441 of 525 functions)
  branches...: 66.6% (395 of 593 branches)

Files changed coverage rate: n/a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mpetersoCode55 mpetersoCode55 Awaiting requested review from mpetersoCode55

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TJ-Everett @excalq