-
Notifications
You must be signed in to change notification settings - Fork 0
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
Cuonglm/issue 1 #2
Conversation
Signed-off-by: Masudur Rahman <[email protected]>
@patcito seems we need to setup CI for the repo? |
Thanks I've set up the CI on https://cloud.drone.io/patcito/gitea |
Do you have access to CI? it seems to be falling not sure why here are the logs attached |
@patcito I'm looking |
@patcito Seems related to this:
|
Sounds like it dues to some modules use |
@patcito Reading this src-d/go-billy#64 (comment), do you think we should use https://github.com/jfontan/go-billy-desfacer or writing our own? |
Could you check if go-billy-desfacer code looks ok? If yes then we could
use it.
…On Mon, Dec 23, 2019, 12:07 PM Cuong Manh Le ***@***.***> wrote:
@patcito <https://github.com/patcito> Reading this src-d/go-billy#64
(comment)
<src-d/go-billy#64 (comment)>, do
you think we should use https://github.com/jfontan/go-billy-desfacer or
writing our own?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=AAAGOQ2PXBPI43UBBWW4BYDQ2CLV3A5CNFSM4J6P3B6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHQ4M5I#issuecomment-568444533>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAGOQ4XTOTOZSZANG7KYSLQ2CLV3ANCNFSM4J6P3B6A>
.
|
I'm creating an PR, seems it's ok since when it jus a wrapper with afero inside. |
Use afero for abstract filesystem layer.
Updates #1