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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split with new files #13
Comments
Hi, thanks for reaching out :) The split works on existing file on the split repository. Maybe a different question... what exactly is your issue with the splitting right now? |
Thanks for your quick response! Basically, I have a repository where I store all my Docker images and I want to split it to take advantage of the build on Docker Hub instead of GitHub Actions. Here are some logs with an empty repository.
Here are some logs with an already initialised repository.
In both cases, my "new" files are not pushed to the split repository. But if I update one of the existing files, the split is correctly handled. 馃槙 |
I don't have any experience with Docker hub splits, so I can't really help there. |
I probably expressed myself poorly, sorry... Let me rephrase. 馃槄 Here is my expected workflow.
I'm not sure to understand as I have a |
I see, I understand this more clearly now. Thanks 馃憤 Just to verify, the https://github.com/origamiphp/docker-blackfire-entrypoint is complete, but This is the directory that should be split? https://github.com/origamiphp/docker-images/tree/main/common/blackfire-entrypoint The GitHub Actions setup seems correct, not sure what is wrong. |
Your two assumptions are correct, and I don't understand this either. 馃檭 I will try to take some time today to investigate further. |
Just a tip: when I try to debug the split, I usually start with single directory and single file. One can get easily lost if there is over 20 files to work with. Often it's some triviality like wrong directory, slashes path, confused setup directory vs repository in the Github action etc. It saves me some sleep-less night to create just 1 repository with 1 file in 1 directory and try to split it into 1 remote repository that should contain just 1 file. |
I've done some more tests, and I really can't see which error in my setup would cause only changes to existing files to be pushed. So, I tested another approach by looking at the instruction that checks if there are any changes. I can reproduce the issue locally with raw Git commands only, as you can see in the following snippet. From my point of view, it would be safer to use something like What do you think?
|
I have no idea how git works on this level. Basically I've hacked this when needed and try to fix if something is broken :) Would you suggest to change this in GitHub action? Would it solve your issue? |
This change would probably allow the script to handle more cases and solve my issue, but I want to do further testing to ensure there are no side effects. I will send you a pull request if my tests are okay, perhaps tomorrow. |
@TomasVotruba It seems to work as expected. 馃コ I have configured the split of 1 repository to 10 other repositories with success. My test scenario involved updating one file ( Just a small note, because I was afraid I had broken something, we'll have to update the documentation before doing the next release. The names of the inputs have indeed been changed in the meantime (source). I could prepare the PR if you want. Thanks again for your work! |
Thanks for testing and sharing real repository where I can see the results easily 馃憤
That would be great 馃檪 |
Hello and thanks for your amazing work (@rectorphp user here)! 馃槈
From what I understand, the split only works with existing files on both sides. If I add a new file, the split does nothing (
[NOTE] No files to change
instead of[NOTE] Adding git commit
). Is it possible to force the indexation of new changes?I assume it's related to this line, but I wanted your point of view before submitting any changes.
The text was updated successfully, but these errors were encountered: