joker1007 / yaml_vault

Yaml file encryption/decryption helper.
MIT License
158 stars 20 forks source link

Change the order of the arguments from_file and initialize #19

Closed morihirok closed 3 years ago

morihirok commented 3 years ago

We have a workflow for AWS KMS that uses Direct Assignment, and it is not working with https://github.com/joker1007/yaml_vault/pull/18. This is because the third argument is now treated as a prefix.

It can be fixed by making the argument order like this PR, which feels right to me. How do you think about that?

joker1007 commented 3 years ago

OK. Thank you for this patch!