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

android add google diff-match-patch #1393

Closed

Conversation

lisong
Copy link

@lisong lisong commented Sep 2, 2018

Add patchedFiles in hotcodepush.json which is use google diff-match-patch tools. this PR is works in android.

@msftclas
Copy link

msftclas commented Sep 2, 2018

CLA assistant check
All CLA requirements met.

@@ -286,4 +293,4 @@ public static void verifyUpdateSignature(String folderPath, String packageHash,

CodePushUtils.log("The update contents succeeded the code signing check.");
}
}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what this ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you mean just the last brace then it is mean that you added new line at the end of file. Possible it is so because your automation code-style generator added this new line.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, it's right, i found the answer when open with sublime text2.

@lisong lisong force-pushed the android-google-diff-match-patch branch from 6204fdc to 6a0298e Compare September 3, 2018 04:12
@alexandergoncharov-zz
Copy link
Contributor

Hi @lisong ,
Thanks for contributing!

Could you please clarify more detailed about your PR? What exactly issue you solved with this PR and what approach have you used?
It can help make review for your PR faster.

Thanks,
Alexander

@lisong
Copy link
Author

lisong commented Sep 4, 2018

@alexandergoncharov thanks for reply! it's for #626 , it's useful features. i wonder if necessary, i will finish the mac and windows platform.

@alexandergoncharov-zz
Copy link
Contributor

@lisong , thanks.

It is logic changes and it require more time for investigate and test it. Sorry for this delay.
Please stay in touch. =)

@lisong
Copy link
Author

lisong commented Sep 5, 2018

I see. it is need change server too.

@Zakeelm
Copy link
Contributor

Zakeelm commented Sep 11, 2018

Hey there!

@lisong Awesome work on this PR. Text file diff would definitely be a wonderful addition to CodePush. Unfortunately, we’re planning on completely re-writing our SDK and service logic in the near future so this is something we don’t think we can commit to at the moment on our end. On the bright side, this is something we actually plan on implementing along with these updates. While I don’t have an ETA on this, it is currently in our roadmap for the future of CodePush 😁

@Zakeelm Zakeelm closed this Sep 11, 2018
@lisong
Copy link
Author

lisong commented Sep 12, 2018

OK

@kufufu9
Copy link

kufufu9 commented Sep 10, 2019

How is it now? @Zakeelm

1 similar comment
@prorigorous
Copy link

How is it now? @Zakeelm

@cani1see
Copy link

Any news on text diff now? what about microsoft's roadmap. @Zakeelm

@hnny09
Copy link

hnny09 commented Apr 1, 2024

How is it now? @Zakeelm
Do you have a plan?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants