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

Running forge from a subdirectory will re-clone services #107

Closed
tristanpemble opened this issue Nov 29, 2017 · 1 comment
Closed

Running forge from a subdirectory will re-clone services #107

tristanpemble opened this issue Nov 29, 2017 · 1 comment

Comments

@tristanpemble
Copy link
Contributor

@tristanpemble tristanpemble commented Nov 29, 2017

demonstration:

λ tree
.
├── README.md
├── forge.yaml
├── k8s
├── service.yaml
└── workspace
    └── customer-portal
        ├── Dockerfile
        ├── k8s
        │   └── deployment.yaml
        └── service.yaml

4 directories, 6 files

λ forge pull
║ [.forge/message-broker] git pull
║ Already up-to-date.
║ [workspace/customer-portal] git pull
║ Already up-to-date.
║ [.forge/admin-portal] git pull
║ Already up-to-date.
║ [.forge/payment-processor] git pull
║ Already up-to-date.
║ git pull
║ Already up-to-date.
║ 42 tasks run, 0 errors
║

λ cd workspace/customer-portal

λ forge pull
║ cloning git@github.com:forge-playground/payment-processor.git->.forge/payment-processor
║ cloning git@github.com:forge-playground/message-broker.git->.forge/message-broker
║ git pull
║ Already up-to-date.
║ [.forge/payment-processor] git pull
║ Already up-to-date.
║ [.forge/message-broker] git pull
║ Already up-to-date.
║ 41 tasks run, 0 errors
║

I was expecting the second forge pull to use the services in ../../.forge

@rhs
Copy link
Contributor

@rhs rhs commented Dec 1, 2017

This should be fixed in 0.3.21

@rhs rhs closed this Dec 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants