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

Add AppVeyor CI config #64

Open
k06a opened this issue Dec 31, 2021 · 6 comments
Open

Add AppVeyor CI config #64

k06a opened this issue Dec 31, 2021 · 6 comments
Labels
CI Continuous integration enhancement

Comments

@k06a
Copy link
Owner

k06a commented Dec 31, 2021

We need AppVeyor build matrix to support at least 2 axis:

  1. Different C++ compilers (gcc, clang, msvc) with different major versions
  2. Different C++ standards: C++11, C++14, C++17, C++20 (boolinq-test14 is not compatible with C++11)
@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 0.03 ETH (110.54 USD @ $3684.59/ETH) attached to it.

@k06a k06a added the CI Continuous integration label Jan 1, 2022
@gitcoinbot
Copy link

gitcoinbot commented Jan 1, 2022

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 264 years, 10 months from now.
Please review their action plans below:

1) tylerferrara has been approved to start work.

Taking a look at your repo 👨‍💻

Learn more on the Gitcoin Issue Details page.

@tylerferrara
Copy link

Adding an appveyor.yml configuration file seems strait-forward and well documented for setting up a build pipeline. According to your travis.yml config, this project only targets linux based x86_64 architectures. Do you expect to expand this with AppVeyor? @k06a

As for testing different versions of C++ (Issue: #63), I will communicate with @mostafabarmshory to prevent duplicate work.

@k06a
Copy link
Owner Author

k06a commented Jan 2, 2022

Interesting question, I believe it should work for x86, x86_64, ARM and ARM64

@tylerferrara
Copy link

Interesting question, I believe it should work for x86, x86_64, ARM and ARM64

We can put this to the test 👍

@tylerferrara
Copy link

tylerferrara commented Jan 4, 2022

Update

Adding AppVeyor support to a fork of this project. This includes GitHub integrations to run builds and tests for every PR.

checks

Once the build configuration supports your requirements (compilers & standards), we can create an AppVeyor project for this repo, using the same appveyor.yml.

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

No branches or pull requests

3 participants