Closed lutzroeder closed 6 years ago
I don't think we should include the full bundle in the bower package. That includes all of the dependencies twice if I'm not mistaken. Happy to have someone with more bower experience review and clarify.
On Dec 27, 2017 10:22 PM, "Lutz Roeder" notifications@github.com wrote:
You can view, comment on, or merge this pull request online at: https://github.com/dagrejs/dagre-d3/pull/299 Commit Summary Include packaged versions in bower File Changes M bower.json https://github.com/dagrejs/dagre-d3/pull/299/files#diff-0 (7) Patch Links: https://github.com/dagrejs/dagre-d3/pull/299.patch https://github.com/dagrejs/dagre-d3/pull/299.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dagrejs/dagre-d3/pull/299, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAt9qCFSUmB-stLMEFmefnSBpDQSjBaks5tEzOJgaJpZM4ROC0o .
You can view, comment on, or merge this pull request online at:
https://github.com/dagrejs/dagre-d3/pull/299 Commit Summary
File Changes
Patch Links:
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dagrejs/dagre-d3/pull/299, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAt9qCFSUmB-stLMEFmefnSBpDQSjBaks5tEzOJgaJpZM4ROC0o .
I don't think we should include the full bundle in the bower package. That includes all of the dependencies twice if I'm not mistaken. Happy to have someone with more bower experience review and clarify.
On Dec 27, 2017 10:22 PM, "Lutz Roeder" notifications@github.com wrote: