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

Update module name to match the repo name #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

caitlinelfring
Copy link

@caitlinelfring caitlinelfring commented Mar 19, 2022

Related to get-woke/woke#181, the replace directive in woke's go.mod is causing errors when installing using go install https://github.com/get-woke/woke/blob/e2c8222a50cbaa3177311e84bad6216639a6c8bb/go.mod#L49. See https://go.dev/doc/go-get-install-deprecation.

I'd like to remove the replace module and use the module directly, but need the module name to match the import name, otherwise I get the following error in go mod tidy:

	github.com/inclusive-dev-tools/go-git/v5/plumbing/format/gitignore: github.com/inclusive-dev-tools/go-git/v5@v5.4.4: parsing go.mod:
	module declares its path as: github.com/go-git/go-git/v5
	        but was required as: github.com/inclusive-dev-tools/go-git/v5

The context around using replace is here.

This change will require a new tag, and I'll also need to update woke to use this tag. In the meantime, I've forked this repo to the get-woke org and am updating woke to use that fork. Would prefer to go back to this fork once this change has been merged. Thanks!!

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

Successfully merging this pull request may close these issues.

1 participant