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

Remove Epoch helper class #1567

Merged
merged 3 commits into from Sep 13, 2018
Merged

Remove Epoch helper class #1567

merged 3 commits into from Sep 13, 2018

Conversation

@bording
Copy link
Member

@bording bording commented Apr 18, 2018

Now that we're targeting netstandard2.0, we can rely on the built in DateTimeOffset helper methods, ToUnixTimeSeconds and FromUnixTimeSeconds instead.

@bording bording requested a review from ethomson Apr 18, 2018
@bording
Copy link
Member Author

@bording bording commented Apr 18, 2018

@ethomson I think something might be wrong with the AppVeyor setup. If you take a look at the details, it actually says one test failed, but it's still reporting everything as green...

@ethomson
Copy link
Member

@ethomson ethomson commented Apr 18, 2018

I think something might be wrong with the AppVeyor setup. If you take a look at the details, it actually says one test failed, but it's still reporting everything as green...

Haha, that's delightful:

- ps: |
    Foreach ($runner in 'xunit.console.exe','xunit.console.x86.exe')
    {
      If ($Env:SHOULD_RUN_COVERALLS -eq $True -and $Env:publish_on_success -eq $True)
      {
        .\packages\OpenCover\tools\OpenCover.Console.exe `
          -register:user `
          "-target:""$Env:userprofile\.nuget\packages\xunit.runner.console\2.3.1\tools\net452\$runner""" `
          "-targetargs:""$Env:APPVEYOR_BUILD_FOLDER\bin\LibGit2Sharp.Tests\Release\net461\LibGit2Sharp.Tests.dll"" -noshadow" `
          "-filter:+[LibGit2Sharp]* -[LibGit2Sharp.Tests]*" `
          -hideskipped:All `
          -output:opencoverCoverage.xml
      }
      ElseIf ($Env:SHOULD_RUN_COVERITY_ANALYSIS -eq $False)
      {
        & "$Env:userprofile\.nuget\packages\xunit.runner.console\2.3.1\tools\net452\$runner" `
            "$Env:APPVEYOR_BUILD_FOLDER\bin\LibGit2Sharp.Tests\Release\net461\LibGit2Sharp.Tests.dll" -noshadow
      }
    }

Maybe the return code is not being bubbled up? I'll poke around with this when I have some time.

@bording
Copy link
Member Author

@bording bording commented Aug 27, 2018

Re-ran the AppVeyor tests on this one, and they all passed this time.

@ethomson ethomson merged commit 321ac06 into master Sep 13, 2018
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@bording bording deleted the remove-epoch branch Sep 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.