Refine CI install and test checks

Author: eggpig

.github/workflows/pr_code_check.yaml

@@ -3,9 +3,11 @@ name: PR code review
 on:
   workflow_dispatch:
   pull_request:
+    branches:
+      - pybind11
   push:
     branches:
-      - master
+      - pybind11
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
@@ -44,6 +46,5 @@ jobs:
     - name: Build and install
       run: pip install --verbose .
 
-    - name: Test
-      run: python tests/test.py
-
+    - name: Smoke test
+      run: python -c "import os, tempfile; os.chdir(tempfile.gettempdir()); import easygraph as eg; G = eg.Graph(); G.add_edge(1, 2); assert G.number_of_nodes() == 2; assert G.number_of_edges() == 1"

.github/workflows/test.yaml

@@ -1,6 +1,13 @@
 name: Test the package with pytest
 
-on: [push, pull_request, workflow_dispatch]
+on:
+  workflow_dispatch:
+  pull_request:
+    branches:
+      - pybind11
+  push:
+    branches:
+      - pybind11
 
 jobs:
   pytest:
@@ -12,12 +19,12 @@ jobs:
         # ubuntu 22.04 has deprecated python 3.6
         python-version: [ "3.8", "3.9", "3.10","3.11", "3.12"]
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           submodules: recursive
 
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
 
@@ -36,4 +43,4 @@ jobs:
 
       - name: Test with pytest
         run: |
-          pytest --disable-warnings --ignore=cpp_easygraph
+          pytest easygraph --disable-warnings

setup.py

@@ -47,6 +47,9 @@ def build_extension(self, ext: CMakeExtension) -> None:
         cmake_args = [
             f"-DCMAKE_LIBRARY_OUTPUT_DIRECTORY={extdir}{os.sep}",
             f"-DPYTHON_EXECUTABLE={sys.executable}",
+            f"-DPython_EXECUTABLE={sys.executable}",
+            f"-DPython3_EXECUTABLE={sys.executable}",
+            "-DPYBIND11_FINDPYTHON=ON",
             f"-DCMAKE_BUILD_TYPE={cfg}",  # not used on MSVC, but no harm
             f"-DEASYGRAPH_ENABLE_GPU={'ON' if enable_gpu else 'OFF'}",
         ]