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

Does not work with submodules..? #29

Open
shanewignall opened this issue Aug 9, 2017 · 2 comments · May be fixed by #54
Open

Does not work with submodules..? #29

shanewignall opened this issue Aug 9, 2017 · 2 comments · May be fixed by #54
Labels

Comments

@shanewignall
Copy link

shanewignall commented Aug 9, 2017

I've tried committing a submodule with one small file in it with the file size hook enabled with the following configuration:

Include: .*
Size: 500000000

But I get an error like the following:

git -c diff.mnemonicprefix=false -c core.quotepath=false -c credential.helper=manager-st push -v --set-upstream origin feature/windowsUI:feature/windowsUI
POST git-receive-pack (1029 bytes)
remote: Hook org.christiangalsterer.stash.filehooks.plugin.hook.FileSizeHook failed. Error:
remote: '/usr/bin/git cat-file --batch-check' exited with code 0 saying: error: unable to find df2451e48dda98c8a0fe4827ed7565ebe841643a
Pushing to http://[email protected]/git/scm/fswg/fsw.git
To http://mydomain.com/git/scm/fswg/fsw.git
! [remote rejected] feature/windowsUI -> feature/windowsUI (pre-receive hook declined)
error: failed to push some refs to 'http://[email protected]/git/scm/fswg/fsw.git'

Is this a known issue, or am I missing something?

Thanks!

@christiangalsterer
Copy link
Owner

I never used submodules so far. So it might be that it is not working.
Which Bitbucket and add-on version to you use?

@shanewignall
Copy link
Author

Bitbucket: 4.11.2
File Hooks Plugin: 2.4.0

@fengxx fengxx linked a pull request May 5, 2019 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants