Skip to content

Conversation

@Francisco-Gamino
Copy link
Contributor

Fixing test pass.

}
}

internal class TestDependencyManager2 : DependencyManager
Copy link
Contributor

Choose a reason for hiding this comment

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

Normally, I would recommend coming up with a better class name here (perhaps something referring to the idea of throwing GetLatestModuleVersion. However, I'm in the process of refactoring this code, and this name will not matter soon, so I will not insist on renaming this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good. Thanks @AnatoliB.

@Francisco-Gamino Francisco-Gamino merged commit 8001606 into Azure:dev Jun 21, 2019
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