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

Remove jquery jasmine thus jquery dependency #11

Closed
wants to merge 2 commits into from
Closed

Remove jquery jasmine thus jquery dependency #11

wants to merge 2 commits into from

Conversation

williscool
Copy link
Contributor

re #9

If only for the benefit of having one less library in there to let get stale in there

William Harris added 2 commits April 24, 2013 13:41
jasmine.Fixtures.prototype.createContainer_ = function(html) 
in lib/jasmine-jquery.js assumes there is a body tag to append 

html to
its easy to add it to an individual project by adding a line like

EnvJasmine.loadGlobal(EnvJasmine.testDir + "lib/jasmine-jquery.js");

to you test.dependencies.js

not needed in the plugin though
@williscool
Copy link
Contributor Author

one extra commit got in there but all it does it wrap the runner javascript in a body tag so jasmine jquery's loadfixtures function will work properly by default.

Let me know if I should take it out and and make another branch with jus the one commit

@williscool
Copy link
Contributor Author

awesome guys!

@guapodero
Copy link

thank you

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.

2 participants