Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Setup vanity import paths #33

Closed
rakyll opened this issue Oct 9, 2017 · 4 comments
Closed

Setup vanity import paths #33

rakyll opened this issue Oct 9, 2017 · 4 comments
Assignees
@rakyll
Labels
api-breaking stats tags trace
Milestone
R1

Comments

@rakyll
Copy link
Contributor

rakyll commented Oct 9, 2017

To reduce to dependencies to a specific git provider and to enhance the the current long and noisy import path, setup a vanity URL for the package and enforce the vanity URL everywhere. The vanity URL is either going to be "go.opencensus.io" or "opencensus.io/go".
@rakyll rakyll self-assigned this Oct 9, 2017
@rakyll
Copy link
Contributor Author

rakyll commented Oct 10, 2017

We are going with go.opencensus.io.

/cc @tags07 @mtwo

@rakyll
Copy link
Contributor Author

rakyll commented Oct 31, 2017

go get go.opencensus.io finally works!

I will document how to deploy the vanity URL server before closing this issue.

@bogdandrutu
Copy link
Contributor

Thanks @rakyll.

@bogdandrutu
Copy link
Contributor

Probably also to track the update of the documentation and examples to use the vanity import in this issue.

rakyll added a commit to rakyll/opencensus-go that referenced this issue Oct 31, 2017
@rakyll
Enforce the new vanity import path
402acb6 
Fixes census-instrumentation#33.
@rakyll rakyll mentioned this issue Oct 31, 2017
Merged
rakyll added a commit to rakyll/opencensus-go that referenced this issue Oct 31, 2017
@rakyll
Enforce the new vanity import path
ee58636 
Fixes census-instrumentation#33.
rakyll added a commit to rakyll/opencensus-go that referenced this issue Nov 1, 2017
@rakyll
Enforce the new vanity import path
1da998c 
Fixes census-instrumentation#33.
@rakyll rakyll closed this as completed in 34bb52b Nov 1, 2017
@rakyll rakyll added this to the R1 milestone Dec 12, 2017
bogdandrutu pushed a commit to bogdandrutu/opencensus-go that referenced this issue Feb 15, 2019
@odeke-em
vendor: remove vendoring by dep
0aa9963 
Firstly removing vendor/ and dep
which will be followed up by a change
switching to using modules. For now
this change enables Travis CI to build.
Could have actually just updated dependencies
using dep but that'd be an extra change and
we are already revamping this repository.

Also while here fix Stackdriver import path:
This is necessary to change to be bundled with removing
dep so that Travis CI can build as the Stackdriver
exporter was migrated to:
    "contrib.go.opencensus.io/exporter/stackdriver"

Updates census-instrumentation#33
bogdandrutu pushed a commit to bogdandrutu/opencensus-go that referenced this issue Feb 15, 2019
@odeke-em
vendor: using go mod
82aed60 
This change is the final half to supplement
PR census-instrumentation#40 which removed vendoring by dep. This
now uses `go mod`.

Fixes census-instrumentation#33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rakyll rakyll

Labels
api-breaking stats tags trace
Projects
None yet
Milestone
R1
Development

No branches or pull requests
2 participants
@rakyll @bogdandrutu