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

Merge with libphonenumber-csharp? #3

Open
ArashMotamedi opened this issue Feb 7, 2017 · 5 comments
Open

Merge with libphonenumber-csharp? #3

ArashMotamedi opened this issue Feb 7, 2017 · 5 comments
Assignees

Comments

@ArashMotamedi
Copy link

Hello and thanks for this port.

Any chance you'd be able to merge your code with the other repo, libphonenumber-csharp? It seems Aidan who's been maintaining that code hasn't had a chance to do a .Net Core port (Issue 6), and if this library merges with that one and publish through Nuget, it'd be great!

Thanks,
Arash

@brunoblank
Copy link

Awesome work!

The source needs to be updated (the xml files, they are from june-2016).

I agree with Arash, the best option is to merge it back to the other repo.

Thanks
Bruno

@jarroda
Copy link
Owner

jarroda commented Mar 6, 2017

Yeah I originally forked this from libphonenumber-csharp with the intention of merging it back in. I was worried the original project wasn't being maintained, but it seems like it is so thats good. I'll try to make some time within the next week or so to update this and get a PR submitted.

@RichiCoder1
Copy link

We just ran into this. Let me know if you'd like any help.

@jarroda
Copy link
Owner

jarroda commented Mar 11, 2017

I just realized that I actually based my fork on a different repo (there are apparently several of these out there). Mine was based on https://github.com/erezak/libphonenumber-csharp, not https://github.com/aidanbebbington/libphonenumber-csharp/.

The latter definitely seems to be the only one that is getting metadata updates, so I'll just manually port it instead. I think I'll just drop my current codebase and switch this repo to aidanbebbington's fork.

I ported the main PhoneNumbers project last night, so the next phase is to see about the test project.

@jarroda jarroda self-assigned this Mar 11, 2017
@jarroda
Copy link
Owner

jarroda commented Mar 11, 2017

Done. twcclegg/libphonenumber-csharp#22

Though I had to create a new fork based on the other repo, so any further work I do will be over there. https://github.com/jarroda/libphonenumber-csharp

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

No branches or pull requests

4 participants