Skip to content

feat: add skip option for ABI lint validation#263

Merged
lcastillo-ledger merged 4 commits intomainfrom
pr/lint-skip-abi-validation-upstream
Mar 5, 2026
Merged

feat: add skip option for ABI lint validation#263
lcastillo-ledger merged 4 commits intomainfrom
pr/lint-skip-abi-validation-upstream

Conversation

@lcastillo-ledger
Copy link
Collaborator

Summary

  • add a --skip-abi-validation option to erc7730 lint and thread it through the lint pipeline
  • keep non-ABI lint checks active while making lint runs more reliable for offline and local development
  • update lint-related tests and descriptor discovery/normalization helpers to keep registry-based tests stable

Test plan

  • run pytest tests/lint/test_lint.py --skip-abi-validation
  • run pytest tests/test_main.py --skip-abi-validation
  • run full test suite in CI

lcastillo-ledger and others added 2 commits March 4, 2026 20:27
Add a CLI and test flag to skip external Etherscan ABI validation during linting so local and offline runs remain stable and faster while keeping all other lint checks active.
@lcastillo-ledger lcastillo-ledger requested a review from a team as a code owner March 5, 2026 14:07
@github-actions github-actions bot added documentation Improvements or additions to documentation sources labels Mar 5, 2026
Keep registry descriptor collection logic unchanged while formatting comprehensions to satisfy Ruff E501 and formatter checks in CI.
Resync submodules and update gitlinks to the latest remote commits so CI uses current registry descriptors.
@lcastillo-ledger lcastillo-ledger merged commit e1a68bd into main Mar 5, 2026
10 checks passed
@lcastillo-ledger lcastillo-ledger deleted the pr/lint-skip-abi-validation-upstream branch March 5, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation sources

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants