Skip to content

Enforce testing of secure password credentials #1117

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 24, 2015

Conversation

Therzok
Copy link
Member

@Therzok Therzok commented Jun 24, 2015

Fixes #1114

@nulltoken
Copy link
Member

@Therzok Wow, you're fast!

Just a question regarding your commit message. Were you willing to start it with Ensure rather than Encore?

@nulltoken
Copy link
Member

or Enforce?

@Therzok Therzok force-pushed the therzok-secure-string branch from 272a776 to e3d6d33 Compare June 24, 2015 08:42
@Therzok Therzok changed the title Encore testing of secure password credentials Enforce testing of secure password credentials Jun 24, 2015
@Therzok
Copy link
Member Author

Therzok commented Jun 24, 2015

Note to self: Never code before morning coffee.

@Therzok Therzok force-pushed the therzok-secure-string branch from e3d6d33 to a2fe304 Compare June 24, 2015 08:42
@Therzok Therzok force-pushed the therzok-secure-string branch from a2fe304 to 43565ce Compare June 24, 2015 08:43
@Therzok
Copy link
Member Author

Therzok commented Jun 24, 2015

Fixed coding styles issues too, I think it's good to go once CI tests it.

@nulltoken nulltoken added this to the v0.22 milestone Jun 24, 2015
nulltoken added a commit that referenced this pull request Jun 24, 2015
Enforce testing of secure password credentials
@nulltoken nulltoken merged commit 0a72962 into vNext Jun 24, 2015
@nulltoken nulltoken deleted the therzok-secure-string branch June 24, 2015 09:26
@nulltoken
Copy link
Member

❤️

@rubberduck203
Copy link
Contributor

Wow. Fast it right. I didn't even get a chance to see @nulltoken's cc before this was done. =;)-

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.

3 participants