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

refactor: add prerequisites for makefile targets involving docker-compose #1476

Merged

Conversation

pgebal
Copy link
Contributor

@pgebal pgebal commented Apr 17, 2020

Closes #1272

It does not solve the issue in general. It solves it only for make commands involving docker-compose with default docker-compose file.

@pgebal pgebal force-pushed the pgebal/fix_dependency_on_local_child_chain_env_in_makefile branch from 371d701 to 6b4cfa2 Compare April 17, 2020 12:32
@pgebal pgebal requested a review from kevsul April 17, 2020 12:32
@coveralls
Copy link

coveralls commented Apr 17, 2020

Coverage Status

Coverage remained the same at 77.581% when pulling cb0d89d on pgebal/fix_dependency_on_local_child_chain_env_in_makefile into 00cf991 on master.

@pgebal pgebal force-pushed the pgebal/fix_dependency_on_local_child_chain_env_in_makefile branch from 6b4cfa2 to 487ac43 Compare April 17, 2020 12:49
@pgebal pgebal marked this pull request as ready for review April 17, 2020 15:43
@pgebal pgebal requested a review from InoMurko April 17, 2020 15:43
Makefile Show resolved Hide resolved
@pgebal pgebal force-pushed the pgebal/fix_dependency_on_local_child_chain_env_in_makefile branch from 487ac43 to 6d764e5 Compare April 17, 2020 16:22
@pgebal pgebal merged commit 6a26336 into master Apr 20, 2020
@pgebal pgebal deleted the pgebal/fix_dependency_on_local_child_chain_env_in_makefile branch April 20, 2020 14:21
@unnawut unnawut added the chore Technical work that does not affect service behaviour label Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Technical work that does not affect service behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docker dependency on localchain_contract_addresses.env
5 participants