Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

fix(std-fpvm): Allow non-const fn with mut ref#1057

Merged
clabby merged 1 commit intomainfrom
cl/lint-fix
Feb 19, 2025
Merged

fix(std-fpvm): Allow non-const fn with mut ref#1057
clabby merged 1 commit intomainfrom
cl/lint-fix

Conversation

@clabby
Copy link
Contributor

@clabby clabby commented Feb 19, 2025

Overview

Fixes the lint routine for the cannon-builder image. These images use the nightly toolchain, rather than the MSRV, to build.

Also loosens the requirements for the build images' platforms, since they're now distributed as cross-platform builds. Rejoice, Apple laptop users :D

@clabby clabby added A-ci Area: CI K-fix Kind: fix labels Feb 19, 2025
@clabby clabby self-assigned this Feb 19, 2025
@codecov
Copy link

codecov bot commented Feb 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.9%. Comparing base (12a17eb) to head (0f96504).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@refcell refcell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice

@clabby clabby enabled auto-merge February 19, 2025 19:25
@clabby clabby added this pull request to the merge queue Feb 19, 2025
Merged via the queue into main with commit cf8aa2b Feb 19, 2025
16 checks passed
@clabby clabby deleted the cl/lint-fix branch February 19, 2025 19:38
This was referenced Feb 19, 2025
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A-ci Area: CI K-fix Kind: fix

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants