Skip to content

Resolve forge linter issues, remove solhint, remove unused files#297

Open
drinkcoffee wants to merge 3 commits into
mainfrom
peter/resolve-linter-warnings
Open

Resolve forge linter issues, remove solhint, remove unused files#297
drinkcoffee wants to merge 3 commits into
mainfrom
peter/resolve-linter-warnings

Conversation

@drinkcoffee
Copy link
Copy Markdown
Contributor

@drinkcoffee drinkcoffee commented May 22, 2026 •
edited
Loading

This PR:

  • Removes Immutable X related files. This was done to reduce the number of warnings to resolve.
  • Removes some unused files related to SeaPort.
  • Resolves all forge linter issues
  • Removes solhint
  • Formats code using "forge fmt"
  • Updates copyright notices to 2026.

@drinkcoffee drinkcoffee changed the title [DRAFT] Resolve forge linter issues, remove solhint, remove unused files Resolve forge linter issues, remove solhint, remove unused files May 22, 2026
@drinkcoffee drinkcoffee marked this pull request as ready for review May 22, 2026 07:43
@drinkcoffee drinkcoffee requested review from a team as code owners May 22, 2026 07:43
@drinkcoffee drinkcoffee requested review from ermyas, lfportal and rytimx May 22, 2026 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ermyas ermyas Awaiting requested review from ermyas

@lfportal lfportal Awaiting requested review from lfportal

@rytimx rytimx Awaiting requested review from rytimx

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@drinkcoffee