Skip to content
This repository was archived by the owner on Apr 11, 2021. It is now read-only.

Add conditionally mounted submodules#97

Open
karlfloersch wants to merge 7 commits intomasterfrom
feat/conditional_mount
Open

Add conditionally mounted submodules#97
karlfloersch wants to merge 7 commits intomasterfrom
feat/conditional_mount

Conversation

@karlfloersch
Copy link
Copy Markdown
Contributor

This PR addresses https://github.com/ethereum-optimism/roadmap/issues/660 by making it so that the -l flag requires specific packages to be passed through. This prevents the anti-pattern in this repo where you feel you have to always build local all of the packages to test.

TODO:
-[ ] Determine what to do about make 😭 I deleted everything that isn't building because I got frustrated about not being able to pass through args 😅 but maybe I should re-add the stuff.

  • Update readme

#!/bin/bash

git pull --recurse-submodules
git pull origin master --recurse-submodules
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This needs smarter logic about the current head

.PHONY: all integration-tests deployer geth-l2 batch-submitter data-transport-layer test

pull:
@echo "Pulling & building all remote images..."
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

pull.sh is broken so i don't think we should recommend using it until it is fixed

@karlfloersch karlfloersch marked this pull request as ready for review March 22, 2021 18:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants