Merge pull request #121 from Chia-Network/EL.remove3.8

Remove python 3.8 support and use 3.9 for other tests
Chia-Network · Oct 16, 2024 · df1daaa · df1daaa
2 parents 536c82d + 0c9c083
commit df1daaa
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml
@@ -24,7 +24,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [macos-13, ubuntu-latest, windows-latest]
-        python: [3.8, 3.9, '3.10', 3.11]
+        python: [3.9, '3.10', 3.11]
 
     steps:
     - uses: actions/checkout@v3
@@ -141,7 +141,7 @@ jobs:
 
 
     - name: Verify recompilation of old sources match with new compiler
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: |
         set -x
         # Build cmd line tools
@@ -163,7 +163,7 @@ jobs:
           ./verify_compiler_version.sh ${CLVM_TOOLS_RS_VERSION} && ./activated.py python tools/manage_clvm.py check)
 
     - name: Test Classic command line tools with pytest
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: |
         python -m pip install pytest
         # This script installs the wheel built during this workflow.
@@ -172,7 +172,7 @@ jobs:
         (cd resources/tests/cmdline/tests && py.test cmds_test.py )
 
     - name: Verify recompilation of cl21 sources
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: |
         # We need chia-rs for the following.
         pip install chia-rs
@@ -183,7 +183,7 @@ jobs:
         (cd resources/tests && python check-recompiles.py)
 
     - name: Verify recompilation follows date and modification rules
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: |
         pip install --no-index --find-links wheel/target/wheels/ clvm_tools_rs
         python resources/tests/test-clvm-recompile-behavior.py
@@ -203,22 +203,22 @@ jobs:
         pytest tests
 
     - name: Run tests
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: cargo test --features=fuzz
 
     - name: Exhaustive assign tests
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: cargo test -- --include-ignored assign
 
     - name: Check coverage
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: |
         cargo install grcov
         rustup component add llvm-tools-preview
         python ./resources/coverage/run_coverage.py --require-percent 60
 
     - name: Build alpine wheel via docker
-      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
+      if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.9')
       run: |
         cd resources/alpine && docker build -t clvm-tools-rs-alpine .
         docker run -v ${GITHUB_WORKSPACE}:/root/clvm_tools_rs -t clvm-tools-rs-alpine sh /root/build-alpine.sh