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

add timezone support #17

Merged
merged 2 commits into from
Oct 26, 2021

Conversation

knicola
Copy link
Contributor

@knicola knicola commented Oct 25, 2021

moving timezone support from breejs/bree#126 to breejs/later

Changelog

Added:

  • Timezone support to .setTimeout() and .setInterval()
  • Pkg sinon@^11.1.2 as a dev dependency

@shadowgate15
Copy link
Member

@knicola Let me know if you need any help with this. If you want, I have some time later today to help fix up some of those unrelated tests that are failing.

@knicola
Copy link
Contributor Author

knicola commented Oct 26, 2021

@shadowgate15 If you could have a look that'd be great. The build seems to be timing out right before the coverage report.

I know I forgot to cleanup a sinon spy so I'll take care of that one in a bit.

@shadowgate15
Copy link
Member

I'm going to go ahead and merge this. the issue is a timing related thing, which just makes it a little flaky. thank you for all your hard work!!!

@shadowgate15 shadowgate15 merged commit 5f5ac40 into breejs:master Oct 26, 2021
@knicola knicola deleted the feature/add-timezone-support branch October 26, 2021 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants