-
Notifications
You must be signed in to change notification settings - Fork 22
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
Cannot use pre-git when node_modules is not in the same folder as .git folder #101
Comments
Hmm, how would the fit hook script find the package json file?
…Sent from my iPhone
On Dec 7, 2016, at 12:31, Ripley511 ***@***.***> wrote:
This is quite annoying: I have a project where the repo contains java back-end sources and the front-end sources in a separate folder, like so:
.
|____.git
|____project-backend
|____project-scripts
|____project-ui
| |____node_modules
| |____package.json
|____project-webapp
Is there any way to tell pre-git to look for package.json in a specific folder? If not can we add this feature? I'm up for it :-)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
We have to assume the user pwd can be anywhere from the deepest to the root folder of his git repository. Once we find the root ( You may even find situations where several sub-folders contain a package.json, each with different commands associated to their git-hooks. It might be complicated though not impossible, in my opinion, to run individual scripts for each of them (but this is another subject). |
sure, so breadth first search for |
Here's an idea used by another git-hook package. |
This is quite annoying: I have a project where the repo contains java back-end sources and the front-end sources in a separate folder, like so:
.
|____.git
|____project-backend
|____project-scripts
|____project-ui
| |____node_modules
| |____package.json
|____project-webapp
Is there any way to tell pre-git to look for package.json in a specific folder? If not can we add this feature? I'm up for it :-)
The text was updated successfully, but these errors were encountered: