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

Declare globprovider module path correctly #1276

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

echlebek
Copy link
Collaborator

@echlebek echlebek commented Oct 6, 2023

The globprovider module path did not equal the globprovider package path. This made it unusable with go get, which would complain that the module path and package paths were not equal.

After changing the module path, I fixed up the otelcolbuilder Makefile in order to ensure that it will always import at the latest version correctly.

@echlebek echlebek self-assigned this Oct 6, 2023
@echlebek echlebek requested a review from a team as a code owner October 6, 2023 20:44
Copy link
Contributor

@sumo-drosiek sumo-drosiek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

The globprovider module path did not equal the globprovider package
path. This made it unusable with go get.

After changing the module path, I fixed up the otelcolbuilder Makefile
in order to ensure that it will always import at the latest version
correctly.

Signed-off-by: Eric Chlebek <[email protected]>
@echlebek echlebek force-pushed the echlebek/fix-globprovider-module branch from 2426211 to 8925e26 Compare October 10, 2023 17:55
@echlebek echlebek enabled auto-merge (squash) October 10, 2023 17:56
@echlebek echlebek merged commit 3cfd577 into main Oct 10, 2023
@echlebek echlebek deleted the echlebek/fix-globprovider-module branch October 10, 2023 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants