Skip to content
This repository has been archived by the owner on Jan 20, 2023. It is now read-only.

Remove optional dependencies and add documentation #16

Closed
jtmoulia opened this issue May 12, 2015 · 0 comments
Closed

Remove optional dependencies and add documentation #16

jtmoulia opened this issue May 12, 2015 · 0 comments

Comments

@jtmoulia
Copy link
Contributor

Spell specifies as deps:

  • websocket_client
  • poison
  • msgpax

Since these dependencies are optional, they should be specified in the parent application's dependencies rather than Spell's. There should be documentation for how to add these dependencies to a Mixfile. See how ecto requires the postgres driver dep to be specified by the parent app.
@jtmoulia jtmoulia mentioned this issue Sep 14, 2015
Closed
jtmoulia added a commit to jtmoulia/spell that referenced this issue Sep 18, 2015
@jtmoulia
Document using Spell and specifying dependencies
0964890 
Remove optional dependencies from requirements, requiring the library
user to specify them explicitly.

Resolves MyMedsAndMe#16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jtmoulia