Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small fixes #32

Merged
merged 3 commits into from
Sep 20, 2024
Merged

Small fixes #32

merged 3 commits into from
Sep 20, 2024

Conversation

feuGeneA
Copy link
Contributor

πŸ“ Summary

Small changes, as described in each commit message.

β›± Motivation and Context

I put this repo to use and hit some friction along the way. These changes eliminate the friction that I experienced.

βœ… I have run these commands

  • make lint
  • make test
  • go mod tidy

prior to this, building the image was causing this warning:

```
 1 warning found (use docker --debug to expand):
 - FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 2)
```
this change causes docker to create an additional intermediate layer containing
just the go.mod file and the downloaded dependencies cited therefrom. this
prevents having to re-download the go.mod dependencies on EVERY image rebuild.
more specifically, as long as you don't change the go.mod file, docker will use
the cached layer.
it's used in the Makefile but, prior to this, the README didn't instruct you to
install it.
@metachris
Copy link
Collaborator

thanks

@metachris metachris merged commit c01a101 into flashbots:main Sep 20, 2024
2 checks passed
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.

2 participants