moby / vpnkit

A toolkit for embedding VPN capabilities in your application
Apache License 2.0
1.09k stars 182 forks source link

appveyor: ensure we generate the COMMIT artefact #572

Closed djs55 closed 2 years ago

djs55 commented 2 years ago

Previous builds seemed to have a cached COMMIT file. Dune doesn't have a "PHONY" concept as far as I can see, perhaps it was a mistake to move this code in there.

djs55 commented 2 years ago

I think it's actually fine, the "problem" is that appveyor builds a merge commit, not the PR commit.