-
Notifications
You must be signed in to change notification settings - Fork 798
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
chore: add appveyor ci configuration #60
Conversation
@Tapppi landed this so that we'd kick off an AppVeyor build, looks like we might need to do some work: https://ci.appveyor.com/project/bcoe/standard-version/build/job/dbrpfj3f570n8h1c Perhaps |
Yeah I just noticed that, I'm debugging on my fork, will report shortly. |
Also, the config is missing the final newline 😢 Hurts my soul that I made that mistake. |
So the build finished, with all the logging. Theres a bunch of git warnings about author info not being set, but I guess that doesn't matter much, except for the fact that it's annoying when debugging. The actual error seems to be https://ci.appveyor.com/project/Tapppi/standard-version/build/10/job/ksaf6xchqvhavmwn After debugging I have found that: |
I have a nagging feeling this could be related to the git log --format command used by gitRawCommit including % signs in the format argument and this % sign thing in appveyor appveyor/ci#246 somehow.. 😞 Still not sure though. I'll look into it more when I can. |
@Tapppi tested on Node V6 on OSX, works BTW -- wish AppVeyor had better output. We should look at the npm/npm appveyor config, I think there are some tricks to get better output. |
@bcoe Can you add a badge to README? appveyor config is per user instead of per org... I can point the CI under my name to this repo... Which is weird .. |
@stevemao 👍 will add as soon as we hammer out this weird issue... yeah, appveyor doesn't do orgs too well. |
So I put a debug build together. It basically has a I have no idea how that can be 😞 |
Here is another test from earlier where I tested |
@Tapppi I have quite a few cycles to put into OSS this weekend, so I can dive in and help debug too; if this is still stumping us. |
I forgot to check the last test i put together and, lo and behold, it got the most interesting results so far. For the heck of it I put another debug together, where the The committer dates ( https://ci.appveyor.com/project/Tapppi/standard-version/build/20/job/piotnr784candb99#L592 As far as I can see, this could be either:
I can't find any documentation pointing out that |
I added ci.appveyor.com to this project. Not much changed since #57, removed the
core.autocrlf = true
config. Is there any need for it really?