Skip to content

Add make command for op-rbuilder image and switch to nightly fmt#542

Closed
SozinM wants to merge 4 commits intoflashbots:developfrom
NethermindEth:msozin/op-rbuilder/op-release-makefile
Closed

Add make command for op-rbuilder image and switch to nightly fmt#542
SozinM wants to merge 4 commits intoflashbots:developfrom
NethermindEth:msozin/op-rbuilder/op-release-makefile

Conversation

@SozinM
Copy link
Contributor

@SozinM SozinM commented Apr 7, 2025

Add +nightly for cargo fmt, clippy and fix
Run make fmt

📝 Summary

💡 Motivation and Context


✅ I have completed the following steps:

  • Run make lint
  • Run make test
  • Added tests (if applicable)

Copilot AI review requested due to automatic review settings April 7, 2025 06:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 18 out of 19 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • Makefile: Language not supported

@SozinM SozinM force-pushed the msozin/op-rbuilder/op-release-makefile branch from 06fe4d7 to 2e46d27 Compare April 7, 2025 08:20
@SozinM SozinM requested a review from sukoneck as a code owner April 7, 2025 12:16
@ZanCorDX
Copy link
Collaborator

ZanCorDX commented Apr 8, 2025

Please don't merge until we discuss the "nightly".

@SozinM SozinM force-pushed the msozin/op-rbuilder/op-release-makefile branch from 4771763 to 4464b1e Compare April 9, 2025 13:54
@SozinM SozinM force-pushed the msozin/op-rbuilder/op-release-makefile branch from 4464b1e to 37656e9 Compare April 21, 2025 02:02
@SozinM
Copy link
Contributor Author

SozinM commented Apr 21, 2025

rebased

@SozinM
Copy link
Contributor Author

SozinM commented May 13, 2025

replaced with flashbots/op-rbuilder#27

@SozinM SozinM closed this May 13, 2025
@SozinM SozinM deleted the msozin/op-rbuilder/op-release-makefile branch May 13, 2025 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants