Correctness fixes, batched miller loop, tests, benchmarks

.github/CODEOWNERS

@@ -0,0 +1,4 @@
+# Code owners are automatically requested for review on every pull request.
+# https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners
+
+* @Marchhill

.github/workflows/test.yml

@@ -124,7 +124,11 @@ jobs:
     needs: [build-linux, build-macos, build-windows]
     env:
       BUILD_CONFIG: release
-    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
     steps:
       - name: Check out blst-bindings repository
         uses: actions/checkout@v5
@@ -136,6 +140,7 @@ jobs:
 
       - name: Move artifacts
         working-directory: src/Nethermind.Crypto.Bls/runtimes
+        shell: bash
         run: |
           mv -f linux-arm64/libblst.so linux-arm64/native/libblst.so
           mv -f linux-x64/libblst.so linux-x64/native/libblst.so
@@ -151,20 +156,18 @@ jobs:
         run: dotnet restore
 
       - name: Build
-        working-directory: src/Nethermind.Crypto.Bls
-        run: >
-          dotnet build -c ${{ env.BUILD_CONFIG }} --no-restore
-            ${{ inputs.preview && format('-p:VersionSuffix=preview.{0}', github.run_number) || '' }}
+        working-directory: src
+        run: dotnet build -c ${{ env.BUILD_CONFIG }} --no-restore ${{ inputs.preview && format('-p:VersionSuffix=preview.{0}', github.run_number) || '' }}
 
       - name: Test
         working-directory: src/Nethermind.Crypto.Test
         run: dotnet test -c ${{ env.BUILD_CONFIG }} --no-restore
 
       - name: Publish
-        if: ${{ inputs.publish }}
+        if: ${{ inputs.publish && matrix.os == 'ubuntu-latest' }}
         working-directory: src/Nethermind.Crypto.Bls
         run: |
-          dotnet pack -c ${{ env.BUILD_CONFIG }} --no-build
+          dotnet pack -c ${{ env.BUILD_CONFIG }} --no-build ${{ inputs.preview && format('-p:VersionSuffix=preview.{0}', github.run_number) || '' }}
           dotnet nuget push bin/${{ env.BUILD_CONFIG }}/*.nupkg \
             -k ${{ github.event.inputs.feed == 'Production' && secrets.NUGET_API_KEY || secrets.NUGETTEST_API_KEY }} \
             -s ${{ github.event.inputs.feed == 'Production' && 'https://api.nuget.org/v3/index.json' || 'https://apiint.nugettest.org/v3/index.json' }}

README.md

@@ -4,3 +4,17 @@
 [![Nethermind.Crypto.Bls](https://img.shields.io/nuget/v/Nethermind.Crypto.Bls)](https://www.nuget.org/packages/Nethermind.Crypto.Bls)
 
 C# bindings for the [Supranational blst library](https://github.com/supranational/blst), supporting operations on the BLS12-381 curve and BLS signatures.
+
+## Development
+
+Run the tests:
+
+```sh
+dotnet test src -c release
+```
+
+Run the benchmarks (optionally filtered by class, e.g. `--filter '*MsmBenchmarks*'`):
+
+```sh
+dotnet run --project src/Nethermind.Crypto.Bls.Bench -c release
+```

src/Directory.Build.props

@@ -13,7 +13,7 @@
     <Authors>Nethermind</Authors>
     <Copyright>Demerzel Solutions Limited</Copyright>
     <SourceRevisionId Condition="'$(Commit)' != ''">$(Commit.Substring(0, 8))</SourceRevisionId>
-    <VersionPrefix>1.0.5</VersionPrefix>
+    <VersionPrefix>1.1.0</VersionPrefix>
     <VersionSuffix></VersionSuffix>
   </PropertyGroup>
 

src/Directory.Packages.props

@@ -3,6 +3,7 @@
     <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
   </PropertyGroup>
   <ItemGroup>
+    <PackageVersion Include="BenchmarkDotNet" Version="0.15.8" />
     <PackageVersion Include="coverlet.collector" Version="6.0.3" />
     <PackageVersion Include="FluentAssertions" Version="[7.2.0]" />
     <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />