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
Initial implementation #1
Conversation
b59e21a
to
a62dda5
Compare
a62dda5
to
c1e93b9
Compare
# If you run "mix test --cover", coverage assets end up here. | ||
/cover | ||
|
||
# The directory Mix downloads your dependencies sources to. |
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.
dependencies'
Thanks, @jmodjeska -- I addressed all your feedback. |
|
||
## Installation | ||
|
||
If [available in Hex](https://hex.pm/docs/publish), the package can be installed as: |
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.
This conditional language feels pretty awkward; I assume we'll publish to hex right when we open-source. Given that, I think we can exclude the conditional entirely and assume the package has already been published?
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.
This is the text that mix new
generates. I was planning to update it once I publish.
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.
SGTM!
LGTM! |
Looks clean for open sourcing to me. |
I want to open source our
ProcessTreeDictionary
implementation. This PR is a start of that process. For now, this repo is private until I get permission to make it public.