Skip to content

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.

4 participants