Skip to content

requireJS for version 2.0.0#18

Merged
hilios merged 1 commit intohilios:v2.0.0from
FagnerMartinsBrack:v2-require
Nov 1, 2013
Merged

requireJS for version 2.0.0#18
hilios merged 1 commit intohilios:v2.0.0from
FagnerMartinsBrack:v2-require

Conversation

@FagnerMartinsBrack
Copy link
Copy Markdown
Contributor

Few notes:
Due to some error with npm install -d while installing the qunit grunt plugin I had to downgrade it to the version 1.0.0 to make it work.
I will comment each line explaining the changed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here we create a single test for requireJS instead of running the whole suite, which would be pretty useless once we agree the support is working

@FagnerMartinsBrack
Copy link
Copy Markdown
Contributor Author

After reviewing this I realized it may not be necessary to create a separate file or hide the jquery global.
If I use the same code in the original test file, without import the plugin by the script tag, I may be able to get this to work without creating so many files.

@FagnerMartinsBrack
Copy link
Copy Markdown
Contributor Author

By the way, can't you use #qunit-fixture instead of $dom = $('<div id="clock" />').appendTo('body'); in the tests file?

@FagnerMartinsBrack
Copy link
Copy Markdown
Contributor Author

Everything in this pull request should be used under wtfpl public license

@hilios
Copy link
Copy Markdown
Owner

hilios commented Nov 1, 2013

I agree with you about using the #qunit-fixture, but I will need to create the element on the fly, I will just append it to the right element, and let qunit do the clean-up.

By the way, this is much a cleaner way to test the requirejs support, great job! I will merge this right away.

Thanks,

@FagnerMartinsBrack
Copy link
Copy Markdown
Contributor Author

but I will need to create the element on the fly

I don't understand. Which element?

@hilios
Copy link
Copy Markdown
Owner

hilios commented Nov 1, 2013

Never mind!

hilios added a commit that referenced this pull request Nov 1, 2013
@hilios hilios merged commit 246dcff into hilios:v2.0.0 Nov 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants