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

GitHub Actions config #38

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

sdarwin
Copy link
Contributor

@sdarwin sdarwin commented Mar 3, 2021

GitHub Actions CI script, generated from the .travis.yml file. Please refer to https://github.com/CPPAlliance/githubactions for more information and instructions.

@eldiener
Copy link
Collaborator

eldiener commented Mar 3, 2021

Why would I need this when I already have a travis.yml, an appveyor.yml, and your own drone.star ? Please explain what is accomplished by all this CI testing stuff. I readily admit that I find all this testing stuff confusing and a PITA in trying to understand how all this works or works together.

@sdarwin
Copy link
Contributor Author

sdarwin commented Mar 3, 2021

@eldiener , answered in the Issue: cppalliance/githubactions#1

It is true that the tests are to a great degree redundant. As long as travis, appveyor, and drone are all working, GitHub Actions may be overkill.

But it can't hurt to have two systems running the same test, in case there's a failure, you can compare the results.

@eldiener
Copy link
Collaborator

eldiener commented Mar 4, 2021

I really do not want to have CI running the same set of tests redundantly within different CI testing contexts. I am perfectly willing to defer to you or others as far as what is the most effective CI setup for this library, but I see no benefit of redundancy every time some change is made to the library.

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