Skip to content

Resolve: Update state variables that could be declared immutable as immutable #605#609

Merged
nkrishang merged 3 commits into
mainfrom
nk/fix-slither-immutable-vars
Jan 25, 2024
Merged

Resolve: Update state variables that could be declared immutable as immutable #605#609
nkrishang merged 3 commits into
mainfrom
nk/fix-slither-immutable-vars

Conversation

@nkrishang

Copy link
Copy Markdown
Contributor

Resolves #605

@nkrishang nkrishang enabled auto-merge (squash) January 25, 2024 10:24
@nkrishang nkrishang disabled auto-merge January 25, 2024 10:34
@codecov

codecov Bot commented Jan 25, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4ff7235) 64.21% compared to head (0f36836) 64.21%.
Report is 1 commits behind head on main.

❗ Current head 0f36836 differs from pull request most recent head 0db0431. Consider uploading reports for the commit 0db0431 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #609   +/-   ##
=======================================
  Coverage   64.21%   64.21%           
=======================================
  Files         215      215           
  Lines        6632     6632           
=======================================
  Hits         4259     4259           
  Misses       2373     2373           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nkrishang nkrishang enabled auto-merge (squash) January 25, 2024 10:56
@nkrishang nkrishang merged commit ed8b329 into main Jan 25, 2024
@nkrishang nkrishang deleted the nk/fix-slither-immutable-vars branch January 25, 2024 11:31
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.

Update state variables that could be declared immutable as immutable

1 participant