ci: Add workflow for Windows

Signed-off-by: Julien Jerphanion <[email protected]>

Author: jjerphan

.github/workflows/build_with_conda.yml

@@ -40,10 +40,10 @@ jobs:
         uses: SimenB/[email protected]
         id: cpu-cores
 
-      - name: Install Conda environment from environment_unix.yml
+      - name: Install Conda environment from environment-dev.yml
         uses: mamba-org/[email protected]
         with:
-          environment-file: environment_unix.yml
+          environment-file: environment-dev.yml
           init-shell: >-
             bash
           cache-environment: true
@@ -124,10 +124,10 @@ jobs:
         uses: SimenB/[email protected]
         id: cpu-cores
 
-      - name: Install Conda environment from environment_unix.yml
+      - name: Install Conda environment from environment-dev.yml
         uses: mamba-org/[email protected]
         with:
-          environment-file: environment_unix.yml
+          environment-file: environment-dev.yml
           environment-name: arcticdb
           init-shell: >-
             bash
@@ -175,3 +175,65 @@ jobs:
         env:
           ARCTICDB_USING_CONDA: 1
 
+  windows:
+    runs-on: windows-latest
+    env:
+      ACTIONS_ALLOW_UNSECURE_COMMANDS: true
+    steps:
+      - uses: actions/[email protected]
+        # DONT use recursive submodules checkout to simulate conda feedstock build
+        # with:
+        #   submodules: recursive
+
+      - name: Get number of CPU cores
+        uses: SimenB/[email protected]
+        id: cpu-cores
+
+      - name: Install Conda environment from environment-dev.yml
+        uses: mamba-org/[email protected]
+        with:
+          environment-file: environment-dev.yml
+          init-shell: >-
+            cmd
+          cache-environment: true
+          post-cleanup: 'all'
+
+      - name: Build ArcticDB with conda (ARCTICDB_USING_CONDA=1)
+        shell: cmd {0}
+        run: |
+          # Protocol buffers compilation require not using build isolation.
+          python -m pip install --no-build-isolation --no-deps -v -e .
+        env:
+          ARCTICDB_USING_CONDA: 1
+          ARCTICDB_BUILD_CPP_TESTS: 1
+          ARCTIC_CMAKE_PRESET: windows-conda-release-build
+
+      - name: Build C++ Tests
+        shell: cmd {0}
+        run: |
+          cd cpp\out\windows-conda-release-build
+          msbuild arcticdb_rapidcheck_tests.vcxproj /p:Configuration=Release /p:Platform=x64 /m
+          msbuild test_unit_arcticdb.vcxproj /p:Configuration=Release /p:Platform=x64 /m
+
+      - name: Run C++ Tests
+        shell: cmd {0}
+        run: |
+          cd cpp\out\windows-conda-release-build
+          ctest -C Release -T test
+
+      - name: Install npm
+        uses: actions/[email protected]
+        with:
+          node-version: '16'
+
+      - name: Install azurite
+        shell: cmd {0}
+        run: |
+          npm install -g azurite
+
+      - name: Test with pytest
+        shell: cmd {0}
+        run: |
+          cd python
+          set ARCTICDB_RAND_SEED=%RANDOM%
+          python -m pytest --timeout=3600

docs/mkdocs/docs/technical/contributing.md

@@ -94,10 +94,10 @@ due to a linkage problems with libprotobuf.
 See: https://github.com/man-group/ArcticDB/pull/449**
 
  - [Install `mamba`](https://mamba.readthedocs.io/en/latest/installation.html)
- - Create the `arcticdb` environment from its specification (`environment_unix.yml`):
+ - Create the `arcticdb` environment from its specification (`environment-dev.yml`):
 
   ```bash
-  mamba env create -f environment_unix.yml
+  mamba env create -f environment-dev.yml
   ```
 
  - Activate the `arcticdb` environment (you will need to do this for every new shell session):
@@ -113,7 +113,7 @@ See: https://github.com/man-group/ArcticDB/pull/449**
 For instance:
 
  - for debug build on Linux with mamba and conda-forge, use:
- 
+
   ```bash
   export ARCTICDB_USING_CONDA=1
   cmake -DTEST=off --preset linux-conda-debug cpp
@@ -395,7 +395,7 @@ If possible, a version number should also be added such that future changes in t
 ### API
 
 Any changes to the API (including when exceptions are thrown and the type of the exception) must be weighed up against the change breaking behaviour for existing users.
-Please make this as clear to reviewers as possible by ensuring **API changes are clearly described in the PR description**. 
+Please make this as clear to reviewers as possible by ensuring **API changes are clearly described in the PR description**.
 If the change is breaking, please also ensure that that is appropriately highlighted in the PR description as well.
 This is particularly true of the `NativeVersionStore` API, as this has many users inside Man Group.
 

environment_unix.yml environment-dev.yml

@@ -82,6 +82,8 @@ dependencies:
   - trustme
   - entt
   - psutil
-  - memray 
-  - pytest-memray
-
+  # Use memray and pytest-memray only on Linux and MacOS
+  - sel(linux): memray
+  - sel(linux): pytest-memray
+  - sel(osx): memray
+  - sel(osx): pytest-memray