Skip to content
This repository was archived by the owner on Jun 18, 2024. It is now read-only.

Add America/Fort_Nelson to known OlsonTimeZones#481

Merged
serious6 merged 1 commit into
OfficeDev:masterfrom
easel:fort-nelson-tz
Mar 19, 2016
Merged

Add America/Fort_Nelson to known OlsonTimeZones#481
serious6 merged 1 commit into
OfficeDev:masterfrom
easel:fort-nelson-tz

Conversation

@easel
Copy link
Copy Markdown
Contributor

@easel easel commented Jan 20, 2016

  • Add more descriptive assertion failures to OlsonTimeZoneTest
  • A new timezone for America/Fort_Nelson was added in tzdata2015g, which
    was included in jdk8u71. This timezone had no analog in the TimeZoneUtils
    map, causing the OlsonTimeZoneTest to fail.

@azurecla
Copy link
Copy Markdown

Hi @easel, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, AZPRBOT;

- Add more descriptive assertion failures to OlsonTimeZoneTest
- A new timezone for America/Fort_Nelson was added in tzdata2015g, which
  was included in jdk8u71. This timezone had no analog in the TimeZoneUtils
  map, causing the OlsonTimeZoneTest to fail.
@codecov-io
Copy link
Copy Markdown

Current coverage is 10.43%

Merging #481 into master will increase coverage by +0.01% as of e7140e1

@@            master    #481   diff @@
======================================
  Files          550     550       
  Stmts        20432   20433     +1
  Branches      2625    2625       
  Methods          0       0       
======================================
+ Hit           2131    2132     +1
  Partial        153     153       
  Missed       18148   18148       

Review entire Coverage Diff as of e7140e1

Powered by Codecov. Updated on successful CI builds.

@serious6 serious6 added this to the 2.1 milestone Mar 19, 2016
serious6 added a commit that referenced this pull request Mar 19, 2016
Add America/Fort_Nelson to known OlsonTimeZones
@serious6 serious6 merged commit 7f8793a into OfficeDev:master Mar 19, 2016
@serious6
Copy link
Copy Markdown
Member

@easel thanks for your contribution

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants