Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[SERVER-1247] Create a common members utility #868

Merged
merged 4 commits into from
Apr 16, 2019

Conversation

Cookiezaurs
Copy link
Contributor

@Cookiezaurs Cookiezaurs commented Mar 22, 2019

SERVER-1247 Create a common members utility

Functions moved to members.js. Changes in docs.sh to generate documentation.
SERVER-1336 also fixed.

Functions moved to members.js. Changes in docs.sh to generate documentation.
[SERVER-1336] also fixed.
@ar2rsawseen
Copy link
Member

This pull request introduces 4 alerts when merging 26eaa0a into 1ad8093 - view on LGTM.com

new alerts:

  • 2 for Useless conditional
  • 2 for Use of password hash with insufficient computational effort

Comment posted by LGTM.com

@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@Countly Countly deleted a comment Mar 22, 2019
@ar2rsawseen
Copy link
Member

This pull request introduces 4 alerts when merging 6519a97 into 1ad8093 - view on LGTM.com

new alerts:

  • 2 for Useless conditional
  • 2 for Use of password hash with insufficient computational effort

Comment posted by LGTM.com

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants