Skip to content
This repository was archived by the owner on May 17, 2021. It is now read-only.

Conversation

@Anemy
Copy link
Member

@Anemy Anemy commented Aug 5, 2020

Copy link
Contributor

@lrlna lrlna left a comment

Choose a reason for hiding this comment

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

yessss!

small nit on the if statement, if you're keen.

if (resp.modules && resp.modules.indexOf('enterprise') !== -1) {

// Cover both cases of detecting enterprise module, see SERVER-18099.
if (getMongoDBBuildInfo.isEnterprise(resp)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

i think you can just do this on line 62 above:

enterprise_module: getMongoDBBuildInfo.isEnterprise(resp)

Copy link
Member Author

@Anemy Anemy Aug 5, 2020

Choose a reason for hiding this comment

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

ah ty lol when I'm thinking about other things to code when coding my code goes 🙃

@Anemy Anemy requested a review from lrlna August 5, 2020 10:26
@Anemy Anemy merged commit a3e8ce6 into master Aug 5, 2020
@Anemy Anemy deleted the COMPASS-4260/use-mongodb-build-info branch August 5, 2020 10:38
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