Skip to content
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

Added file encryption #12

Merged
merged 2 commits into from
Aug 24, 2013
Merged

Added file encryption #12

merged 2 commits into from
Aug 24, 2013

Conversation

heisam
Copy link
Contributor

@heisam heisam commented Jul 14, 2013

Hi Mark,

please review and maybe merge my changes to your gem. I'm still new to programming and this is my first pull request to an open source project. So please let me know, if I should add anything.

Kindly, Heiner

@mdp
Copy link
Owner

mdp commented Aug 14, 2013

Looks like a good addition, but it's failing in Ruby 1.8.7 - https://travis-ci.org/mdp/gibberish/jobs/9069646

I think it just needs to explicitly pass the temp filename string

@heisam
Copy link
Contributor Author

heisam commented Aug 23, 2013

Thanks for your feedback. It's fixed now.

mdp added a commit that referenced this pull request Aug 24, 2013
@mdp mdp merged commit ac31887 into mdp:master Aug 24, 2013
@mdp
Copy link
Owner

mdp commented Aug 24, 2013

Thanks Heiner, this is in v1.3.0

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