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

Release: Error in Sizzle version check #1939

Closed
dmethvin opened this Issue Dec 17, 2014 · 6 comments

Comments

Projects
None yet
2 participants
@dmethvin
Member

dmethvin commented Dec 17, 2014

Dies during the check, not sure why. I just backed out 1d93106 fon 1.11-stable and 2.1-stable, but we'll need to to the same or fix it before 3.0 in both master and compat.

Checking Sizzle version...

/var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/node_modules/q/q.js:126
                    throw e;
                          ^
TypeError: Arguments to path.join must be strings
    at path.js:360:15
    at Array.filter (native)
    at Object.exports.join (path.js:358:36)
    at GitHubResolver._checkout (/var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/lib/core/resolvers/GitHubResolver.js:54:21)
    at /var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/lib/core/resolvers/GitResolver.js:69:21
    at _fulfilled (/var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/node_modules/q/q.js:798:54)
    at self.promiseDispatch.done (/var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/node_modules/q/q.js:827:30)
    at Promise.promise.promiseDispatch (/var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/node_modules/q/q.js:760:13)
    at /var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/node_modules/q/q.js:574:44
    at flush (/var/www/jquery-release/__release/repo/node_modules/grunt-bowercopy/node_modules/bower/node_modules/q/q.js:108:17)

@dmethvin dmethvin added this to the 3.0.0 milestone Dec 17, 2014

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Dec 17, 2014

Member

I don't think it likes the version number. However, I'm not sure you'll encounter the same problem on master/compat since we dropped bower.

Member

timmywil commented Dec 17, 2014

I don't think it likes the version number. However, I'm not sure you'll encounter the same problem on master/compat since we dropped bower.

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Dec 17, 2014

Member

Oh, I understand. You won't encounter this problem on master/compat because the Sizzle version there is not funky.

Member

timmywil commented Dec 17, 2014

Oh, I understand. You won't encounter this problem on master/compat because the Sizzle version there is not funky.

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Dec 17, 2014

Member

rversion didn't match so it's passing null. I'll do a fix, but it won't matter.

Member

timmywil commented Dec 17, 2014

rversion didn't match so it's passing null. I'll do a fix, but it won't matter.

@dmethvin

This comment has been minimized.

Show comment
Hide comment
@dmethvin

dmethvin Dec 17, 2014

Member

If that occurs would it be considered an error?

Member

dmethvin commented Dec 17, 2014

If that occurs would it be considered an error?

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Dec 17, 2014

Member

I suppose it would be right now, but it might not be in the future.

Member

timmywil commented Dec 17, 2014

I suppose it would be right now, but it might not be in the future.

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Dec 17, 2014

Member

That is, if we were to publish a version of Sizzle to npm that does not exactly fit the format of x.x.x-optionalWord.

Member

timmywil commented Dec 17, 2014

That is, if we were to publish a version of Sizzle to npm that does not exactly fit the format of x.x.x-optionalWord.

@timmywil timmywil closed this in 78ac753 Dec 17, 2014

@dmethvin dmethvin modified the milestones: 1.12/2.2, 3.0.0 Jan 7, 2016

@cssmagic cssmagic referenced this issue May 18, 2016

Open

jQuery #5

@lock lock bot locked as resolved and limited conversation to collaborators Jun 19, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.