Skip to content

Conversation

jcharum
Copy link
Contributor

@jcharum jcharum commented Jul 22, 2020

Moves the regression test programs into their own directories/packages. This makes Go tooling happier:

  • go test ./... works without any exclusion (see Travis configuration of go test ./... $(go list ./... | grep -v regress/tests)).
  • golangci-lint works without any exclusion.

@jcharum jcharum requested a review from josh-newman July 22, 2020 16:37
@jcharum jcharum mentioned this pull request Jul 22, 2020
Copy link
Contributor

@josh-newman josh-newman left a comment

Choose a reason for hiding this comment

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

Optional: Move to regress/test1, regress/teststream directly. The regress/tests/* hierarchy seems unnecessary.

@jcharum jcharum merged commit 99c2cf2 into grailbio:master Jul 22, 2020
@jcharum jcharum deleted the regress-tests-package branch July 22, 2020 22:39
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