JsonDiffPatch icon indicating copy to clipboard operation
JsonDiffPatch copied to clipboard

Conflict with Tavis.JsonPointer library

Open fhucko opened this issue 5 years ago • 3 comments

We have project that references this library, and Tavis.JsonPointer library, and both libraries contain same class in same namespace. It would be nice if this library used the other library, or changed namespace of JsonPointer in this library. Currently we have to use reference alias + "extern alias" in code.

fhucko avatar Jun 20 '19 07:06 fhucko

Which class?

On Thu, 20 Jun 2019, 08:22 hogan, [email protected] wrote:

We have project that references this library, and Tavis.JsonPointer library, and both libraries contain same class in same namespace. It would be nice if this library used the other library, or changed namespace of JsonPointer in this library. Currently we have to use reference alias + "extern alias" in code.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/mcintyre321/JsonDiffPatch/issues/9?email_source=notifications&email_token=AACDJ6RPUCTUU2YGCL57Q3LP3MV47A5CNFSM4HZQGD3KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G2SRSFA, or mute the thread https://github.com/notifications/unsubscribe-auth/AACDJ6WLXLG5XVHRZO6IMRTP3MV47ANCNFSM4HZQGD3A .

mcintyre321 avatar Jun 20 '19 08:06 mcintyre321

The only one in Tavis namespace in this library. Tavis.JsonPointer.

fhucko avatar Jun 20 '19 08:06 fhucko

I'd be happy to accept a PR renaming the class/namespace, as long as a comment is included.

I'd prefer to reference Tavis.JsonPointer via nuget, but it seems there isn't a netstandard (non beta) package.

On Thu, 20 Jun 2019 at 09:19, hogan [email protected] wrote:

The only one in Tavis namespace in this library. Tavis.JsonPointer.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/mcintyre321/JsonDiffPatch/issues/9?email_source=notifications&email_token=AACDJ6VNS2RI4M3XB5C5PHDP3M4SVA5CNFSM4HZQGD3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYEVHYY#issuecomment-503927779, or mute the thread https://github.com/notifications/unsubscribe-auth/AACDJ6T76QGXDWKN3CFG6A3P3M4SVANCNFSM4HZQGD3A .

mcintyre321 avatar Jun 21 '19 13:06 mcintyre321