Skip to content

Pkg migration#256

Merged
matthiasgeihs merged 4 commits intohyperledger-labs:devfrom
perun-network:pkg-migration
Nov 16, 2021
Merged

Pkg migration#256
matthiasgeihs merged 4 commits intohyperledger-labs:devfrom
perun-network:pkg-migration

Conversation

@ggwpez
Copy link
Copy Markdown
Contributor

@ggwpez ggwpez commented Nov 10, 2021

Signed-off-by: Oliver Tale-Yazdi <oliver@perun.network>
@ggwpez ggwpez force-pushed the pkg-migration branch 3 times, most recently from d99e7fe to 7b60718 Compare November 15, 2021 21:30
@ggwpez ggwpez marked this pull request as ready for review November 15, 2021 21:33
@ggwpez ggwpez requested a review from matthiasgeihs November 15, 2021 21:33
@ggwpez ggwpez mentioned this pull request Nov 15, 2021
Comment thread client/test/doc.go Outdated

// Package test contains testing setup types and functions for package client.
package test // import "perun.network/go-perun/pkg/test"
package test // import "polycry.pt/poly-go/test"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks wrong.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch… seems like the vanity import script is not finding this??

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I expanded the vanity script, it now finds it and reports:
Package has no or wrong vanity import path: ~/go/src/github.com/perun-network/go-perun/client/test, want: perun.network/go-perun/client/test.

Signed-off-by: Oliver Tale-Yazdi <oliver@perun.network>
Signed-off-by: Oliver Tale-Yazdi <oliver@perun.network>
The script now also checks for the correct import path,
not just some.

Signed-off-by: Oliver Tale-Yazdi <oliver@perun.network>
@matthiasgeihs matthiasgeihs merged commit 258bf07 into hyperledger-labs:dev Nov 16, 2021
@matthiasgeihs matthiasgeihs deleted the pkg-migration branch November 16, 2021 12:37
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.

2 participants