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 AppVeyor config #61

Merged
merged 3 commits into from
Sep 9, 2017
Merged

Add AppVeyor config #61

merged 3 commits into from
Sep 9, 2017

Conversation

miltador
Copy link
Contributor

@miltador miltador commented Sep 9, 2017

@wjdp thanks for great tool and blazing fast alternative to html-proofer!
I decided to take a look at what I can do with #18.
Ended up in working AppVeyor config with artifacts being released on tags builds to GitHub.

Here are the build logs: https://ci.appveyor.com/project/miltador/htmltest
And here is my test release of Windows build to proof it's working:
https://github.com/miltador-forks/htmltest/releases/tag/windows-builds

So this is it, please take a look and, if it's fine, consider the next steps:

Closes #18.

@wjdp
Copy link
Owner

wjdp commented Sep 9, 2017

Ooh, fantastic! Thanks for this. Will have a look this weekend and see if we can get this set-up.

@codecov-io
Copy link

codecov-io commented Sep 9, 2017

Codecov Report

Merging #61 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage   95.67%   95.65%   -0.02%     
==========================================
  Files          17       17              
  Lines         902      898       -4     
==========================================
- Hits          863      859       -4     
  Misses         34       34              
  Partials        5        5
Impacted Files Coverage Δ
htmltest/htmltest.go 97.74% <0%> (-0.07%) ⬇️

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 7d4d1ae...9900fad. Read the comment docs.

@wjdp wjdp force-pushed the appveyor branch 3 times, most recently from 2cbe1d8 to 9900fad Compare September 9, 2017 17:59
@wjdp wjdp merged commit 9900fad into wjdp:master Sep 9, 2017
wjdp added a commit that referenced this pull request Sep 9, 2017
Thanks @miltador

I wasn't able to get S3 uploads working. Will spin out into a new issue.
@miltador miltador deleted the appveyor branch September 9, 2017 18:11
This pull request was closed.
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.

3 participants