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

Updated appveyor config #974

Merged
merged 2 commits into from
Nov 7, 2018
Merged

Conversation

siad007
Copy link
Member

@siad007 siad007 commented Oct 21, 2018

Appveyor config needs to be updated to test against windows.
With this changes the test execution under windows is working again see https://ci.appveyor.com/project/siad007/phing
@mrook would you mind to have a look here and make an account on appveyor for phing, so we are able to check, if there would be any problem with phing running on windows systems? Then we could also put a badge on the readme indicating the status?

@siad007 siad007 requested a review from mrook October 21, 2018 13:46
@codecov
Copy link

codecov bot commented Oct 21, 2018

Codecov Report

Merging #974 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             master    #974   +/-   ##
========================================
  Coverage      44.9%   44.9%           
  Complexity     9925    9925           
========================================
  Files           499     499           
  Lines         24053   24053           
========================================
  Hits          10800   10800           
  Misses        13253   13253

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc8b466...9f053e4. Read the comment docs.

@mrook mrook merged commit 31c9a3d into phingofficial:master Nov 7, 2018
@siad007 siad007 deleted the UpdatedAppveyor branch November 10, 2018 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants