Skip to content

remove gotest.tools for testing#32

Closed
thaJeztah wants to merge 1 commit intomoby:masterfrom
thaJeztah:remove_gotest_tools
Closed

remove gotest.tools for testing#32
thaJeztah wants to merge 1 commit intomoby:masterfrom
thaJeztah:remove_gotest_tools

Conversation

@thaJeztah
Copy link
Member

Reduce the number of dependencies for this module.

Reduce the number of dependencies for this module.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member Author

Looks like I had this branch created somewhere in 2020, but never pushed, but I just discovered why; I was waiting for old go versions to be dropped;

Error: ./term_test.go:21:3: t.Cleanup undefined (type *testing.T has no field or method Cleanup)
FAIL	github.com/moby/term [build failed]
?   	github.com/moby/term/windows	[no test files]
FAIL
Error: Process completed with exit code 2.

Let me do so first

@thaJeztah
Copy link
Member Author

Opened #33

Let me include this in #21

@thaJeztah thaJeztah closed this Nov 5, 2022
@thaJeztah thaJeztah deleted the remove_gotest_tools branch November 5, 2022 15:29
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