Reduce number of dependencies

[kyleconroy]

I recently created a new project that used this project.

mkdir new-project
cd new-project
go mod init .
go get github.com/kevinburke/twilio-go

Bringing this library into the project results in the following modules being added to go.mod

module .

go 1.12

require (
	github.com/go-stack/stack v1.8.0 // indirect
	github.com/golang/protobuf v1.3.1 // indirect
	github.com/inconshreveable/log15 v0.0.0-20180818164646-67afb5ed74ec // indirect
	github.com/kevinburke/go-types v0.0.0-20181103050146-31250f49d75a // indirect
	github.com/kevinburke/go.uuid v1.2.0 // indirect
	github.com/kevinburke/rest v0.0.0-20190424050029-77c0c82d0ded // indirect
	github.com/kevinburke/twilio-go v0.0.0-20181202201842-113f30c37433 // indirect
	github.com/mattn/go-colorable v0.1.1 // indirect
	github.com/mattn/go-isatty v0.0.7 // indirect
	github.com/ttacon/builder v0.0.0-20170518171403-c099f663e1c2 // indirect
	github.com/ttacon/libphonenumber v1.0.1 // indirect
	golang.org/x/sync v0.0.0-20190423024810-112230192c58 // indirect
	gopkg.in/mgo.v2 v2.0.0-20180705113604-9856a29383ce // indirect
)

protobuf support appears to come from libphonenumber, which seems reasonable.

Any idea why this requires mgo?

[kgolding]

Ouch - just added this (very useful) repo in a project and it put 3MB on the binary (and that's with -ldflags s -w). Am just looking at how to slim it down.

[kgolding]

Looking like "github.com/ttacon/libphonenumber" is a large culprit. This commit below, moving the PhoneNumber methods out has made a ~40% binary size reduction: kgolding@2382148

[kevinburke]

How come that works? Every Message object should have PhoneNumber types attached

…

On Fri, May 24, 2019 at 4:02 PM Kevin Golding ***@***.***> wrote: Looking lib "github.com/ttacon/libphonenumber" is a large culprit. This commit below, moving the PhoneNumber methods out has made a ~40% binary size reduction: ***@***.*** <kgolding@2382148> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#39?email_source=notifications&email_token=AABZEI2333RVOLD6MGBFM2DPXBXZPA5CNFSM4HMSEP4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGXKGY#issuecomment-495809819>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABZEI5B4X2IGBKDR2ZZV7TPXBXZPANCNFSM4HMSEP4A> .

-- Kevin Burke phone: 925-271-7005 | kevin.burke.dev

[kgolding]

It's only NewPhoneNumber(), and the .Friendly() and .Local() methods that use the libphonenumber dependency. PhoneNumber is just an alias of string.

And these three only seem to be used in the tests :)