Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Add module definitions for test fixtures #2306

Merged
merged 3 commits into from
Mar 2, 2019

Conversation

lggomez
Copy link
Contributor

@lggomez lggomez commented Feb 6, 2019

No description provided.

@lggomez
Copy link
Contributor Author

lggomez commented Feb 6, 2019

Aside from fixtures that contain incomplete code on purpose (completions) the ones that contain working code like diffTestData may be split into individual modules (as they are separate programs)

@@ -0,0 +1 @@
module github.com/microsoft/vscode-go/gofixtures/basetest
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this assume that the user has cloned the vscode-go repo under github.com/microsoft or that doesn't matter?
Sorry, my module knowledge is not up to date :(

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Don't worry, I've struggled myself with this lately (we don't use 1.11 at work yet)

As long as they're not dependencies it doesn't matter which path they have, I believe

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @lggomez

@jhendrixMSFT That is your understanding as well, right?

Copy link
Member

Choose a reason for hiding this comment

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

Sorry for the delay. This is correct, since nobody will be importing these packages the value is solely used as the identity.

@ramya-rao-a ramya-rao-a merged commit e2f8782 into microsoft:master Mar 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants