Update module name to match the repository.#108
Update module name to match the repository.#108creachadair wants to merge 1 commit intoinformalsystems:masterfrom creachadair:mjf/update-module-name
Conversation
Since this module was moved to a new repository, the go tool complains on
import that its declared name doesn't match its requirement spec:
module declares its path as: github.com/interchainio/tm-load-test
but was required as: github.com/informalsystems/tm-load-test
This updates the module declaration and the internal import paths, along with
the build string injection rule in the Makefile.
Since the repository was forwarded, and all the existing issues still work at
the new address, I also updated the changelog for consistency.
thanethomson
left a comment
There was a problem hiding this comment.
Awesome, thanks for this @creachadair!
This repo does need a little TLC at the moment.
Codecov Report
@@ Coverage Diff @@
## master #108 +/- ##
=======================================
Coverage 52.39% 52.39%
=======================================
Files 16 16
Lines 1960 1960
=======================================
Hits 1027 1027
Misses 833 833
Partials 100 100
|
|
Thanks for the review! I think you'll have to merge this, if you want to, since I do not have commit access here. |
|
Ah, I see our repo's configured to only accept signed commits. It won't let me merge. |
Blrgh. I need to go re-figure-out how to do signed commits. I used to have to do this, but it's been a while. |
|
Don't worry, I can redo your changes in a separate PR - it's standard practice for us nowadays. |
In that case, may I go ahead and close this without disrupting your intent? |
Done with #111. Thanks for bringing this to my attention! |
Since this module was moved to a new repository, the go tool complains on
import that its declared name doesn't match its requirement spec:
This updates the module declaration and the internal import paths, along with
the build string injection rule in the Makefile.
Since the repository was forwarded, and all the existing issues still work at
the new address, I also updated the changelog for consistency.
(I ran into this while investigating tendermint/tendermint/issues/7032)