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

Fix import collision error in test file #5

Merged
merged 1 commit into from
Oct 20, 2017

Conversation

wingyplus
Copy link
Contributor

It's happen when run fetch library with go get subcommand

$ go test -v ./...
cpp_compliance_test.go:9:2: case-insensitive import collision: "github.com/bytearena/box2d" and "github.com/ByteArena/box2d"
FAIL	github.com/ByteArena/box2d [setup failed]

because github username use ByteArena not bytearena

It's happen when run fetch library with `go get` subcommand

```
$ go test -v ./...
cpp_compliance_test.go:9:2: case-insensitive import collision: "github.com/bytearena/box2d" and "github.com/ByteArena/box2d"
FAIL	github.com/ByteArena/box2d [setup failed]
```

because github username use `ByteArena` not `bytearena`
Copy link
Member

@xtuc xtuc left a comment

Choose a reason for hiding this comment

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

Thanks for your PR.

I agree, it's better to be consistent with the repo's name. Even if it worked somehow before on Travis and locally. Go is inconsistent or too permissions with the case.

@netgusto make sure you use the capitalized version locally.

@xtuc xtuc merged commit 58acea7 into ByteArena:master Oct 20, 2017
@xtuc
Copy link
Member

xtuc commented Oct 20, 2017

@wingyplus wingyplus deleted the fix-package-name branch October 20, 2017 13:23
fe3dback pushed a commit to fe3dback/box2d that referenced this pull request Aug 30, 2020
Fix import collision error in test file
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.

None yet

2 participants