-
Notifications
You must be signed in to change notification settings - Fork 103
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
Env var option as part of README #57
Conversation
Question. Is it possible to use this setup for all settings? I would like to use it for the port and servername aswell |
Good spot! You are right, we should add this information in the README. Really, we should add it because the library not only supports this feature for What do you think of adding a more general sentence in the README about the support of |
username: "your.name@your.domain", | ||
password: "pa55word", | ||
username: "your.name@your.domain", # or {:system, "SMTP_USERNAME"} | ||
password: "pa55word", # or {:system, "SMTP_PASSWORD"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you use "SMTP_PASS"
here? Or, update "SMTP_PASS"
in the library documentation to use "SMTP_PASSWORD"
?
We can use SMTP_PASSWORD
, the naming is better but we would just like to keep all the example consistent. :)
What do you think about that addition to the README |
Thanks - looks good to me! 👍 If you can also add this sentence to the library documentation and rebase your commits, we will be really happy to merge it |
develop is the default branch on your repository? rebasing won't do anything, unless I am missunderstanding |
Sorry - you are right the default branch is develop. I meant squashing your commits. |
There is a squash and merge option on PRs is there not. |
That's right, I always forget about this feature :) Anyway, we can't merge the PR right now because the build fails. Credo found this issue on your last commit:
|
It's a shame these tools can't make formatting changes. I normally just use the github web interface when contributing to documentation stuff. |
Thanks for the Pull Request :) |
No description provided.