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

Fix the Appveyor build #72

Closed
ndmitchell opened this issue Sep 5, 2019 · 4 comments
Closed

Fix the Appveyor build #72

ndmitchell opened this issue Sep 5, 2019 · 4 comments
Labels

Comments

@ndmitchell
Copy link
Contributor

https://ci.appveyor.com/project/ndmitchell/filepath is failing. The problem is that it doesn't find time on the plan. Trying the same project but renaming filepath to anything else fixes the issue, so it seems to be that by putting filepath in the plan it invalidates time somehow. A Stack bug, but I'm not convinced one they'll necessarily want to fix, as overwriting filepath is an odd thing to do.

@ndmitchell
Copy link
Contributor Author

No progress on this from the Stack side, and it's a bit of an odd case, so I remove the Appveyor script in 2b7adb0.

@Rufflewind
Copy link
Member

FWIW, directory was able to migrate from AppVeyor + Travis CI to a hand-crafted GitHub Actions config: https://github.com/haskell/directory/blob/master/.github/workflows/build.yml

The initial setup took some time to figure out -- the lack of standardization didn't help -- but past the learning curve it's actually pretty convenient and low-maintenance. Not having to deal with two CI services (AppVeyor + Travis CI) was a nice perk.

@ndmitchell
Copy link
Contributor Author

I've moved all my other projects to GitHub actions and agree it's clearly the best way to go. However, I'm in the process of transferring maintainership, so think the new maintainer should have choice of which CI to use, and how to configure it etc: #79

@hasufell hasufell added the CI label Nov 25, 2021
@hasufell
Copy link
Member

#89

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants