Skip to content

[CONFIG] [Github Actions] lint and test actions splitted #710

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 21, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 30 additions & 0 deletions .github/workflows/prettier-json.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
---
name: Prettier JSON Lint

on: # yamllint disable-line rule:truthy
push:
branches: ['main']
pull_request:
# The branches below must be a subset of the branches above
branches: ['main']
workflow_dispatch:

jobs:
json-lint:
name: 'Prettier JSON Lint'
runs-on: ['ubuntu-24.04']
steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4

- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: 24.x

- name: Install Prettier
run: |
npm install -g [email protected]

- name: Prettier LINT / Style JSON
run: >
prettier --check ./src/**/*.json
19 changes: 3 additions & 16 deletions .github/workflows/python.yml → .github/workflows/python-lint.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
name: Python CI Tests
name: Python CI Lint

on: # yamllint disable-line rule:truthy
push:
Expand All @@ -9,12 +9,9 @@ on: # yamllint disable-line rule:truthy
branches: ['main']
workflow_dispatch:

env:
LOG_LEVEL: INFO

jobs:
build:
name: 'Run CI'
lint:
name: 'Python CI Lint'
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -52,13 +49,3 @@ jobs:
- name: Styling (autopep8)
run: |
python3 -m autopep8 --diff --recursive --exit-code --verbose .
- name: Test
run: >
coverage run -m pytest --verbose
-o log_cli=true
--log-cli-level=INFO
src/

- name: Coverage
run: |
coverage report
45 changes: 45 additions & 0 deletions .github/workflows/python-test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
---
name: Python CI Test

on: # yamllint disable-line rule:truthy
push:
branches: ['main']
pull_request:
# The branches below must be a subset of the branches above
branches: ['main']
workflow_dispatch:

env:
LOG_LEVEL: INFO

jobs:
test:
name: 'Python CI Test'
strategy:
fail-fast: false
matrix:
os: ['ubuntu-24.04', 'macos-14', 'windows-2022']
python: ['3.10', '3.11', '3.12']
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4

- name: Setup Python
uses: actions/setup-python@master
with:
python-version: ${{ matrix.python }}

- name: Install
run: |
pip3 install -r requirements.txt

- name: Test
run: >
coverage run -m pytest --verbose
-o log_cli=true
--log-cli-level=INFO
src/

- name: Coverage
run: |
coverage report
14 changes: 6 additions & 8 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -67,28 +67,26 @@ dependencies:
${PACKAGE_TOOL} install -r requirements.txt
@echo "################################################################################"

lint/json:
prettier --check ./src/**/*.json

lint/markdown:
markdownlint '**/*.md' --ignore node_modules && echo '✔ Your code looks good.'

lint/yaml:
yamllint --stric . && echo '✔ Your code looks good.'

lint: lint/markdown lint/yaml test/styling test/static
lint: lint/markdown lint/yaml lint/json test/styling test/static

test/static: dependencies
${RUNTIME_TOOL} -m pylint --verbose --recursive yes src/
${RUNTIME_TOOL} -m flake8 --verbose src/
${RUNTIME_TOOL} -m pyright --verbose src/


test/styling/json: dependencies
prettier --check ./src/**/*.json

test/styling/sources: dependencies
test/styling: dependencies
${RUNTIME_TOOL} -m pycodestyle --statistics src/
${RUNTIME_TOOL} -m autopep8 --diff --recursive --exit-code --verbose .

test/styling: dependencies test/styling/sources test/styling/json

format/json:
prettier --write ./src/**/*.json

Expand Down