docs: consolidate #204
docs: consolidate #204
Conversation
| "encoding/json" | ||
| "fmt" | ||
|
|
||
| "github.com/twilio/twilio-go" |
There was a problem hiding this comment.
I
| "github.com/twilio/twilio-go" | |
| twilio "github.com/twilio/twilio-go" |
We refer to e.g. "twilio.NewRestClient" below without explaining where the 'twilio' comes from
There was a problem hiding this comment.
Doesn't seem to be necessary, at least based on the behavior of my IDE. If it's a Go convention, sure, but I'd rather not edit every sample unless it's warranted.
There was a problem hiding this comment.
Oh, it's not necessary for sure I just want to avoid confusion. Most of the time the package name matches the import but when they don't match it's not always obvious where it's coming from.
There was a problem hiding this comment.
I always rename these in my own programs for clarity FYI, I'm not just saying this because it's a tutorial, and it doesn't increase the line count.
stern-shawn
force-pushed
the
docs/consolidate
branch
from
1257740
to
f3ef0ab
Compare
stern-shawn
force-pushed
the
docs/consolidate
branch
from
cd5ecf5
to
07154e6
Compare
|
Fixes
As part of the docs changes, I'm consolidating docs content into GitHub so that we can more closely version our content.