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

Release 1.2.5 #1085

Closed
wants to merge 4 commits into from
Closed

Release 1.2.5 #1085

wants to merge 4 commits into from

Conversation

ariya
Copy link
Contributor

@ariya ariya commented Feb 28, 2015

Backport implicit octal scanning, release housekeeping.

@ariya
Copy link
Contributor Author

ariya commented Feb 28, 2015

@scottgonzalez The CLA checker did flag this again.

@scottgonzalez
Copy link
Member

I'm guessing the part of the script that fetches updated signatures isn't working. I've filed jquery/jquery-license#22 and I'll dig in to this on Monday.

@mikesherov
Copy link
Member

Just curious... Why the JSLint skip?

@mikesherov
Copy link
Member

👍

@ariya
Copy link
Contributor Author

ariya commented Mar 1, 2015

Why the JSLint skip?

For some unknown reason, Node.js >= 0.11 fails to run it properly.

ariya added a commit that referenced this pull request Mar 1, 2015
@ariya
Copy link
Contributor Author

ariya commented Mar 1, 2015

Landed in 1.2 branch!

@ariya ariya closed this Mar 1, 2015
@scottgonzalez
Copy link
Member

@ariya There was in fact a problem with the signatures not getting updated while the server was running. I've just pushed an update that fixes that problem so future PRs shouldn't have that issue.

@ariya ariya deleted the release-1.2.5 branch March 2, 2015 16:45
@ariya
Copy link
Contributor Author

ariya commented Mar 2, 2015

@scottgonzalez Thanks!

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.

None yet

4 participants