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

feat: remove need for pg within mev-inspect env #344

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

0xr3x
Copy link

@0xr3x 0xr3x commented Sep 18, 2023

  • instructions for running without kubernetes ('monolithic mode')

  • added docker instructions

  • chore: remove pgsql as hard dependency

  • chore: update deps

  • docs: updated docs to remove local pg engine for docker install

  • docs: reword docs

  • ci: update poetry source

  • fix: refactor tests for mypy

  • fix: search miner for eth2

  • feat: improve eth2 miner fn

  • refactor: unnecessary comma

  • test: add miner generation tests


What does this PR do?

This PR goes beyond the existing monolythic PR #324, updating some of the dependencies to improve compatibility.

Testing

What testing was performed to verify this works? Unit tests are a big plus!

Checklist before merging

  • Read the contributing guide
  • Installed and ran pre-commit hooks
  • All tests pass with ./mev test

* instructions for running without kubernetes ('monolithic mode')

* added docker instructions

* chore: remove pgsql as hard dependency

* chore: update deps

* docs: updated docs to remove local pg engine for docker install

* docs: reword docs

* ci: update poetry source

* fix: refactor tests for mypy

* fix: search miner for eth2

* feat: improve eth2 miner fn

* refactor: unnecessary comma

* test: add miner generation tests

---------

Co-authored-by: pintail <[email protected]>
@0xr3x 0xr3x marked this pull request as ready for review September 18, 2023 17:28
@taarushv taarushv merged commit b3438d7 into flashbots:main Oct 11, 2023
1 check 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