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

#137 Add support for absolute paths #139

Merged
merged 2 commits into from
Apr 12, 2022

Conversation

ruurtjan
Copy link
Contributor

@ruurtjan ruurtjan commented Apr 4, 2022

Solves #137

The code would have to be refactored if you'd like this logic covered by tests. Let me know if you have suggestions, or feel free to commit it yourself. If you think it's okay like this, then we can merge without further changes.

Copy link
Owner

@Philippus Philippus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you mind adding some information to the readme about this feature?

@ruurtjan
Copy link
Contributor Author

Something like that?

@Philippus Philippus merged commit c4be8b9 into Philippus:main Apr 12, 2022
@Philippus
Copy link
Owner

I've released v3.0.0 yesterday, can you try it out? The group id has changed, you can now add sbt-dotenv like this:
addSbtPlugin("nl.gn0s1s" % "sbt-dotenv" % "3.0.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