boolinq icon indicating copy to clipboard operation
boolinq copied to clipboard

Add AppVeyor CI config

Open k06a opened this issue 2 years ago • 6 comments

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)

k06a avatar Dec 31 '21 23:12 k06a

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.

gitcoinbot avatar Dec 31 '21 23:12 gitcoinbot

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.

gitcoinbot avatar Jan 01 '22 03:01 gitcoinbot

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.

tylerferrara avatar Jan 02 '22 17:01 tylerferrara

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

k06a avatar Jan 02 '22 17:01 k06a

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

We can put this to the test 👍

tylerferrara avatar Jan 02 '22 18:01 tylerferrara

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.

tylerferrara avatar Jan 04 '22 14:01 tylerferrara