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

feat: replace with snapshot/*.json diffing #5

Merged
merged 5 commits into from
Jan 29, 2025
Merged

Conversation

boredland
Copy link
Contributor

@boredland boredland commented Jan 29, 2025

  • will act as a drop-in replacement for repos with snapshot/*.json present

@boredland boredland force-pushed the feat/snapshot-compare branch 14 times, most recently from d837faa to 072eb3d Compare January 29, 2025 16:51
@bgd-labs bgd-labs deleted a comment from github-actions bot Jan 29, 2025
@boredland boredland force-pushed the feat/snapshot-compare branch from 072eb3d to 76716cd Compare January 29, 2025 16:54
@bgd-labs bgd-labs deleted a comment from github-actions bot Jan 29, 2025
@boredland boredland force-pushed the feat/snapshot-compare branch from 76716cd to 24bfe2d Compare January 29, 2025 16:55
Copy link

github-actions bot commented Jan 29, 2025

♻️ Gas report

Path Value
mocks/some.json
full amount; receiver: ->enableCollateral ↑7% (+10001) 154,838
full amount; sender: ->disableCollateral; ↓10% (-10000) 93,274
partial amount; sender: collateralEnabled; ↓100% (-103180) 123
full amount; sender: ->disableCollateral; receiver: ->enableCollateral 145,016
🔕 Unchanged
Path Value
mocks/some.json
full amount; sender: ->disableCollateral; receiver: dirty, ->enableCollateral 133,059
full amount; sender: collateralDisabled 103,095
partial amount; sender: collateralDisabled; 103,095
partial amount; sender: collateralDisabled; receiver: ->enableCollateral 144,837
partial amount; sender: collateralEnabled; receiver: ->enableCollateral 145,045
mocks/someOther.json
full amount; receiver: ->enableCollateral 144,837
full amount; sender: ->disableCollateral; 103,274
full amount; sender: ->disableCollateral; receiver: ->enableCollateral 145,016
full amount; sender: ->disableCollateral; receiver: dirty, ->enableCollateral 133,059
full amount; sender: collateralDisabled 103,095
partial amount; sender: collateralDisabled; 103,095
partial amount; sender: collateralDisabled; receiver: ->enableCollateral 144,837
partial amount; sender: collateralEnabled; 103,303
partial amount; sender: collateralEnabled; receiver: ->enableCollateral 145,045

@boredland boredland force-pushed the feat/snapshot-compare branch from 24bfe2d to 008f3c6 Compare January 29, 2025 16:56
@bgd-labs bgd-labs deleted a comment from github-actions bot Jan 29, 2025
@boredland boredland force-pushed the feat/snapshot-compare branch from 008f3c6 to 62155a3 Compare January 29, 2025 17:11
@boredland boredland force-pushed the feat/snapshot-compare branch 3 times, most recently from 5d43cdb to ca1d17e Compare January 29, 2025 17:27
@boredland boredland force-pushed the feat/snapshot-compare branch from ca1d17e to 64bb0aa Compare January 29, 2025 17:31
@boredland boredland force-pushed the feat/snapshot-compare branch from b227246 to fd355a8 Compare January 29, 2025 18:41
@boredland boredland changed the title feat: add snapshot diffing feat: replace with snapshot/*.json diffing Jan 29, 2025
@boredland boredland merged commit d58e195 into main Jan 29, 2025
1 check passed
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