Skip to content

chore(lint): add require-exhaustive-init linter to structs (1/n) - #3445

Closed
TropicalDog17 wants to merge 7 commits into
OffchainLabs:masterfrom
TropicalDog17:chore/add-linter
Closed

chore(lint): add require-exhaustive-init linter to structs (1/n)#3445
TropicalDog17 wants to merge 7 commits into
OffchainLabs:masterfrom
TropicalDog17:chore/add-linter

Conversation

@TropicalDog17

Copy link
Copy Markdown
Contributor

Description

Addresses #3121

Add the linter to packages: arbstate, arbutil, blocks_reexecutor

@CLAassistant

CLAassistant commented Aug 3, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@eljobe
eljobe self-requested a review August 4, 2025 12:58
@eljobe eljobe self-assigned this Aug 4, 2025
eljobe
eljobe previously approved these changes Aug 4, 2025
@eljobe
eljobe enabled auto-merge August 4, 2025 12:59
@TropicalDog17

Copy link
Copy Markdown
Contributor Author

I will go fix the lint issues @eljobe, wasn't able to fully setup the repo locally for linting

auto-merge was automatically disabled August 4, 2025 17:41

Head branch was pushed to by a user without write access

@TropicalDog17

Copy link
Copy Markdown
Contributor Author

make lint return no error locally.

@TropicalDog17
TropicalDog17 requested a review from eljobe August 4, 2025 17:42
@eljobe
eljobe enabled auto-merge August 5, 2025 08:42
@eljobe

eljobe commented Aug 5, 2025

Copy link
Copy Markdown
Member

@TropicalDog17, looks like there are now some failing tests.

@eljobe eljobe assigned TropicalDog17 and unassigned eljobe Aug 5, 2025
@pmikolajczyk41

Copy link
Copy Markdown
Member

continued in #3575

auto-merge was automatically disabled September 4, 2025 13:08

Pull request was closed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants