Skip to content
This repository has been archived by the owner on May 9, 2022. It is now read-only.

Fix "scribe-common" dependency #3

Closed
wants to merge 2 commits into from

Conversation

TooTallNate
Copy link

The package on npm does not depend on scribe-common, however the source code for this plugin attempts to load scribe-common/element.

In addition, lodash-amd was declared as a dependency but was not used in this plugin, so removed that as a dependency.

A re-publish to npm after this would be much appreciated! 🍻

lodash is not used in this plugin.
Matches the `bower.json` file this way.
@samvasko
Copy link

I have still trouble after doing this manually myself. I also had to change
define(['scribe-common/element'], function (element) {
to
define(['scribe-common/src/element'], function (element) {

And it kinda leaves me puzzled how to actually fix this.

@OliverJAsh
Copy link

@Bliker See guardian/scribe-common#2 (comment).

@TooTallNate Merge conflict!

@OliverJAsh
Copy link

Ping @TooTallNate

@rrees rrees self-assigned this Jan 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants