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

ci: Remove appveyor. #388

Merged
merged 1 commit into from
Feb 10, 2024
Merged

ci: Remove appveyor. #388

merged 1 commit into from
Feb 10, 2024

Conversation

waywardmonkeys
Copy link
Contributor

This is now handled by the GitHub Actions CI.

This is now handled by the GitHub Actions CI.
@waywardmonkeys
Copy link
Contributor Author

image

I think you want to remove those 3 somehow.

@recp recp merged commit 125002c into recp:master Feb 10, 2024
73 of 74 checks passed
@recp
Copy link
Owner

recp commented Feb 10, 2024

@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 waywardmonkeys deleted the remove-appveyor branch February 10, 2024 23:35
@waywardmonkeys
Copy link
Contributor Author

@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
Copy link
Owner

recp commented Feb 11, 2024

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

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.

2 participants