search

recp / cglm

📽 Highly Optimized 2D / 3D Graphics Math (glm) for C
MIT License
2.26k stars 229 forks source link

ci: Remove appveyor. #388

Closed waywardmonkeys closed 7 months ago

waywardmonkeys commented 7 months ago

This is now handled by the GitHub Actions CI.

waywardmonkeys commented 7 months ago
image

I think you want to remove those 3 somehow.

recp commented 7 months ago

@waywardmonkeys thanks for your contributions, the PR is merged 🚀

I think you want to remove those 3 somehow.

Travis should be dropped but I think Codacy or better alternatives can still exist to be aware of code quality. Measuring coverage is also crucial because engines, games, apps... will use math and all lines, operations must be tested well on all available platforms with all possible configuration e.g. SSE, AVX, NEON... if something is wrong we should focus on DEBUG our work/software and trust the math and the math library.

waywardmonkeys commented 7 months ago

@recp Yes, it is good to have them, but they aren't working / configured. Can add codacy back if it is set up later rather than have CI runs marked as having pending tasks. Appveyor stuff is now marking checks as having failed for commits to master.

recp commented 7 months ago

@waywardmonkeys I have just removed Travis-CI, AppVeyor and Codacy webhooks, will try to make Codacy work later then can be added later.