cowprotocol / cowprotocol/contracts

ci: make gas snapshot workflow strict

Open
#152 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

E:5.2 Contracts migrations infra
Dominant language
Solidity
Stars
158
Forks
56
Avg merge
9h 40m
Merged PRs (30d)
1

Description

Problem

During the foundry migration, the gas snapshot workflow was relaxed to be lenient as the migration took place. Once this is done, the leniency should be disabled.

Solution

Make gas workflow strict.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the gas snapshot workflow and review how it was made lenient during the Foundry migration. Restore strict behavior and verify that the gas snapshot workflow now rejects unexpected gas changes.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, solidity
Domain
blockchain, ci-cd
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.