Skip to content
This repository was archived by the owner on May 20, 2025. It is now read-only.

android add google diff-match-patch - #1393

Closed
lisong wants to merge 3 commits into
microsoft:masterfrom
lisong:android-google-diff-match-patch
Closed

android add google diff-match-patch#1393
lisong wants to merge 3 commits into
microsoft:masterfrom
lisong:android-google-diff-match-patch

Conversation

@lisong

@lisong lisong commented Sep 2, 2018

Copy link
Copy Markdown

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

@msftclas

Microsoft Contribution License Agreements (msftclas) commented Sep 2, 2018

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.


CodePushUtils.log("The update contents succeeded the code signing check.");
}
} No newline at end of file

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

lisong commented Sep 4, 2018

Copy link
Copy Markdown
Author

Alexander Goncharov (@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
Copy Markdown
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

lisong commented Sep 5, 2018

Copy link
Copy Markdown
Author

I see. it is need change server too.

@Zakeelm

Copy link
Copy Markdown
Contributor

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 😁

@lisong

lisong commented Sep 12, 2018

Copy link
Copy Markdown
Author

OK

@kufufu9

kufufu9 commented Sep 10, 2019

Copy link
Copy Markdown

How is it now? Zakeel M. (@Zakeelm)

1 similar comment
@prorigorous

Copy link
Copy Markdown

How is it now? Zakeel M. (@Zakeelm)

@cani1see

Copy link
Copy Markdown

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

@hnny09

Copy link
Copy Markdown

How is it now? Zakeel M. (@Zakeelm)
Do you have a plan?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants