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

Testing/sauce labs integration #1374

Merged
merged 23 commits into from Dec 29, 2013

Conversation

Projects
None yet
3 participants
@ichernev
Contributor

ichernev commented Dec 25, 2013

This is a continuation of #1301

@alexstrat any comments? I already used this setup to test the latest release, so in terms of usability -- it works (haven't tested the travis build hooks though).

A few tests fail right now (IE8 and Safari), these are being worked on, so ignore them :)

@ichernev

This comment has been minimized.

Contributor

ichernev commented Dec 25, 2013

TODO:

  • fix the broken tests, or somehow make the build succeed with broken tests (don't think it is a good idea)
  • the sauce launcher is broken, it thinks it prints to a terminal, and if the terminal is too narrow (or there is no terminal) it prints status of each browser a few times per second, which floods the logs like crazy -- travis is complaining
@alexstrat

This comment has been minimized.

Contributor

alexstrat commented Dec 27, 2013

Looks good!

Concerning the terminal issue, you should probably change the reporter to something more Travis friendly like the dot reporter.

@icambron

This comment has been minimized.

Member

icambron commented Dec 28, 2013

This is so awesome. Is this ready to merge?

@ichernev

This comment has been minimized.

Contributor

ichernev commented Dec 28, 2013

If you're happy I'll merge it. I want to take one last look.
On Dec 27, 2013 6:23 PM, "Isaac Cambron" notifications@github.com wrote:

This is so awesome. Is this ready to merge?


Reply to this email directly or view it on GitHubhttps://github.com//pull/1374#issuecomment-31288406
.

// it doesn't need Z at the end.
var d = new Date(leftZeroFill(year, 6, true) + '-01-01').getUTCDay(),
daysToAdd, dayOfYear;
var d = makeUTCDate(year, 0, 1).getUTCDay(), daysToAdd, dayOfYear;

This comment has been minimized.

@icambron

icambron Dec 28, 2013

Member

what happened here? Seems like a rollback?

This comment has been minimized.

@ichernev

ichernev Dec 28, 2013

Contributor

Ie doesn't parse long years at all. When I wrote the original code the
makeDate functions didn't work with small years (or for some unknown stupid
reason) I wrote the string parsing inside... Stupid idea.
On Dec 27, 2013 9:12 PM, "Isaac Cambron" notifications@github.com wrote:

In moment.js:

@@ -1497,11 +1497,7 @@

 //http://en.wikipedia.org/wiki/ISO_week_date#Calculating_a_date_given_the_year.2C_week_number_and_weekday
 function dayOfYearFromWeeks(year, week, weekday, firstDayOfWeekOfYear, firstDayOfWeek) {
  •    // The only solid way to create an iso date from year is to use
    
  •    // a string format (Date.UTC handles only years > 1900). Don't ask why
    
  •    // it doesn't need Z at the end.
    
  •    var d = new Date(leftZeroFill(year, 6, true) + '-01-01').getUTCDay(),
    
  •        daysToAdd, dayOfYear;
    
  •    var d = makeUTCDate(year, 0, 1).getUTCDay(), daysToAdd, dayOfYear;
    

what happened here?


Reply to this email directly or view it on GitHubhttps://github.com//pull/1374/files#r8575464
.

@icambron

This comment has been minimized.

Member

icambron commented Dec 28, 2013

OK, I'm happy with this. Good work!

@ichernev

This comment has been minimized.

Contributor

ichernev commented Dec 29, 2013

We have a bug with weekYear/week/weekday setters. Lets fight it in another issue/PR.

ichernev added a commit that referenced this pull request Dec 29, 2013

Merge pull request #1374 from ichernev/testing/sauce-labs-integration
Testing/sauce labs integration

@changelog
@Section misc
@description Run tests on multiple browser/os combos via SauceLabs and Travis

@ichernev ichernev merged commit ef0365e into moment:develop Dec 29, 2013

1 check failed

default The Travis CI build failed
Details

@theurere theurere referenced this pull request Jul 1, 2014

Merged

Update moment.js to 2.7.0 #58

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