Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Package: Add jQuery as explicit dependency, with min and max range #1790

Closed
wants to merge 1 commit into from
Closed

Package: Add jQuery as explicit dependency, with min and max range #1790

wants to merge 1 commit into from

Conversation

jzaefferer
Copy link
Member

1.7.0 is our minimum supported version, 3.x.x our maximum.

Closes gh-1779

@Bajix this is supposed to replace your PR, making the dependency a version range, and without the whitespace changes. Would that work for you?

1.7.0 is our minimum supported version, 3.x.x our maximum.

Closes gh-1779
@jsf-clabot
Copy link

jsf-clabot commented Jan 25, 2017

CLA assistant check
All committers have signed the CLA.

@@ -50,7 +50,9 @@
"scripts": {
"test": "grunt"
},
"dependencies": {},
"dependencies": {
"jquery": ">= 1.7.0 < 4.0.0"
Copy link
Member

@scottgonzalez scottgonzalez Jan 25, 2017

Choose a reason for hiding this comment

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

I haven't tested locally, but all of the docs for npm and semver show no spaces between the operator and the version, so just to be safe, let's drop the spaces after the operators.

@jzaefferer jzaefferer deleted the jquery-dependency branch January 25, 2017 21:39
@Bajix
Copy link

Bajix commented Jan 25, 2017

@jzaefferer 👍

All that really matters is that it's represented

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants