enable github actions for PRs on all branches

Major changes
- update github actions to run building and type check actions for PRs on **all branches**
- separate actions for building and sonarcloud
- delete unused github action documentation.yml

.github/workflows/build.yml

@@ -12,9 +12,6 @@ on:
       - setup.py
       - .github/workflows/build.yml
   pull_request:
-    branches:
-      - main
-      - dev
     paths:
       - bin/**
       - src/**
@@ -89,37 +86,3 @@ jobs:
           install-nplinker-deps --run-on-github
       - name: Run unit tests
         run: pytest -v
-
-  sonarcloud:
-    name: SonarCloud
-    needs: smoke_test
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: Set up Python
-        uses: actions/setup-python@v3
-        with:
-          python-version: '3.10'
-      - name: Python info
-        shell: bash -l {0}
-        run: |
-          which python3
-          python3 --version
-      - name: Upgrade pip and setuptools
-        run: |
-          python3 -m pip install --upgrade pip setuptools wheel
-      - name: Install nplinker and its dependencies
-        run: |
-          python3 -m pip install .[dev]
-          install-nplinker-deps --run-on-github
-      - name: Check style against standards using ruff
-        run: ruff .
-      - name: Run unit tests with coverage
-        run: pytest --cov --cov-report term --cov-report xml --junitxml=xunit-result.xml tests/
-      - name: Correct coverage paths
-        run: sed -i "s+$PWD/++g" coverage.xml
-      - name: SonarCloud Scan
-        uses: SonarSource/sonarcloud-github-action@master
-        env:
-          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN }}
-          SONAR_TOKEN: ${{secrets.SONAR_TOKEN }}

.github/workflows/sonar-cloud.yml

@@ -0,0 +1,63 @@
+name: Check format and coverage, then upload to SonarCloud
+
+on:
+  push:
+    branches:
+      - main
+      - dev
+    paths:
+      - bin/**
+      - src/**
+      - tests/**
+      - setup.py
+      - .github/workflows/build.yml
+  pull_request:
+    branches:
+      - main
+      - dev
+    paths:
+      - bin/**
+      - src/**
+      - tests/**
+      - setup.py
+      - .github/workflows/build.yml
+    types:
+      - opened
+      - reopened
+      - synchronize
+      - ready_for_review
+
+jobs:
+
+  sonarcloud:
+    name: SonarCloud
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Python
+        uses: actions/setup-python@v3
+        with:
+          python-version: '3.10'
+      - name: Python info
+        shell: bash -l {0}
+        run: |
+          which python3
+          python3 --version
+      - name: Upgrade pip and setuptools
+        run: |
+          python3 -m pip install --upgrade pip setuptools wheel
+      - name: Install nplinker and its dependencies
+        run: |
+          python3 -m pip install .[dev]
+          install-nplinker-deps --run-on-github
+      - name: Check style against standards using ruff
+        run: ruff check .
+      - name: Run unit tests with coverage
+        run: pytest --cov --cov-report term --cov-report xml --junitxml=xunit-result.xml tests/unit
+      - name: Correct coverage paths
+        run: sed -i "s+$PWD/++g" coverage.xml
+      - name: SonarCloud Scan
+        uses: SonarSource/sonarcloud-github-action@master
+        env:
+          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN }}
+          SONAR_TOKEN: ${{secrets.SONAR_TOKEN }}

.github/workflows/typing-check.yml

@@ -9,9 +9,6 @@ on:
       - src/**
       - tests/**
   pull_request:
-    branches:
-      - main
-      - dev
     paths:
       - src/**
       - tests/**

pyproject.toml

@@ -67,6 +67,8 @@ ignore_missing_imports = true
 [tool.ruff]
 target-version = "py310"
 line-length = 100
+
+[tool.ruff.lint]
 select = [
     "D", # pydocstyle
     "E", # pycodestyle (error)
@@ -100,7 +102,7 @@ fixable = ["A", "B", "C", "D", "E", "F", "I"]
 ignore-init-module-imports = true
 
 
-[tool.ruff.isort]
+[tool.ruff.lint.isort]
 known-first-party = ["nplinker"]
 force-single-line = true
 lines-after-imports = 2