Skip to content

Commit

Permalink
Merge branch 'main' into pin_in_lock
Browse files Browse the repository at this point in the history
  • Loading branch information
altendky committed Nov 26, 2024
2 parents 972884d + 7fe2373 commit bd639ec
Show file tree
Hide file tree
Showing 670 changed files with 15,636 additions and 12,624 deletions.
10 changes: 0 additions & 10 deletions .flake8

This file was deleted.

1 change: 0 additions & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
* @Chia-Network/required-reviewers
/.github/**/* @Chia-Network/actions-reviewers
/PRETTY_GOOD_PRACTICES.md @altendky @Chia-Network/required-reviewers
/pylintrc @altendky @Chia-Network/required-reviewers
/tests/ether.py @altendky @Chia-Network/required-reviewers
7 changes: 1 addition & 6 deletions .github/workflows/check_wheel_availability.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,13 +43,8 @@ jobs:
matrix: arm
- name: Intel
matrix: intel
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
exclude:
- os:
matrix: macos
arch:
matrix: arm
python-version: "3.8"
- os:
matrix: windows
arch:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/dependency-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,5 @@ jobs:
- name: "Dependency Review"
uses: actions/[email protected]
with:
allow-dependencies-licenses: pkg:pypi/pylint, pkg:pypi/pyinstaller
allow-dependencies-licenses: pkg:pypi/pyinstaller
deny-licenses: AGPL-1.0-only, AGPL-1.0-or-later, AGPL-1.0-or-later, AGPL-3.0-or-later, GPL-1.0-only, GPL-1.0-or-later, GPL-2.0-only, GPL-2.0-or-later, GPL-3.0-only, GPL-3.0-or-later
1 change: 0 additions & 1 deletion .github/workflows/pre-commit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ jobs:
- name: Intel
matrix: intel
python:
- major_dot_minor: "3.8"
- major_dot_minor: "3.9"
- major_dot_minor: "3.10"
- major_dot_minor: "3.11"
Expand Down
54 changes: 40 additions & 14 deletions .github/workflows/test-install-scripts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,34 +22,55 @@ concurrency:

jobs:
test_scripts:
name: Test Install Scripts - ${{ matrix.development.name }} - ${{ matrix.editable.name }}
runs-on: ${{ matrix.os.runs-on }}
name: Native ${{ matrix.os.emoji }} ${{ matrix.arch.emoji }} ${{ matrix.development.name }} - ${{ matrix.editable.name }}
runs-on: ${{ matrix.os.runs-on[matrix.arch.matrix] }}
strategy:
fail-fast: false
matrix:
python:
- major-dot-minor: "3.10"
os:
- runs-on: macos-latest
matrix: macos-arm
- runs-on: macos-13
matrix: macos-intel
- runs-on: ubuntu-latest
- name: Linux
emoji: 🐧
runs-on:
arm: [Linux, ARM64]
intel: ubuntu-latest
matrix: linux
- runs-on: windows-latest
- name: macOS
emoji: 🍎
runs-on:
arm: macos-latest
intel: macos-13
matrix: macos
- name: Windows
emoji: 🪟
runs-on:
intel: windows-latest
matrix: windows
arch:
- name: ARM
emoji: 💪
matrix: arm
- name: Intel
emoji: 🌀
matrix: intel
development:
- name: Non-development
- name: Non-dev
value: false
- name: Development
- name: Dev
value: true
editable:
- name: Non-editable
- name: Non-edit
value: false
matrix: non-editable
- name: Editable
- name: Edit
value: true
matrix: editable
exclude:
- os:
matrix: windows
arch:
matrix: arm

steps:
- name: Checkout Code
Expand Down Expand Up @@ -112,7 +133,7 @@ jobs:
[ "$POST_VERSION" != "shooby-doowah" -a "$PRE_VERSION" = "$POST_VERSION" ]
test_scripts_in_docker:
name: Test Install Scripts ${{ matrix.distribution.name }} ${{ matrix.arch.name }}
name: Docker ${{ matrix.distribution.name }} ${{ matrix.arch.name }}
runs-on: ${{ matrix.os.runs-on[matrix.arch.matrix] }}
container: ${{ matrix.distribution.url }}
strategy:
Expand Down Expand Up @@ -220,7 +241,12 @@ jobs:
apt-get install --yes software-properties-common
add-apt-repository --yes ppa:git-core/ppa
apt-get --yes update
apt-get install --yes git lsb-release sudo python3-venv
apt-get install --yes git lsb-release sudo
MINIMUM=3.9
if ! apt satisfy --yes "python3-venv (>= ${MINIMUM})"
then
apt-get install --yes python${MINIMUM}-venv
fi
- name: Add safe git directory
run: git config --global --add safe.directory "$GITHUB_WORKSPACE"
Expand Down
23 changes: 7 additions & 16 deletions .github/workflows/test-single.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ name: test
on:
workflow_call:
inputs:
emoji:
os-emoji:
required: true
type: string
matrix:
Expand All @@ -35,6 +35,9 @@ on:
arch:
required: true
type: string
arch-emoji:
required: true
type: string
collect-junit:
required: false
type: boolean
Expand All @@ -50,31 +53,23 @@ defaults:

jobs:
test:
name: ${{ matrix.os.emoji }} ${{ matrix.configuration.name }} - ${{ matrix.python.name }}
name: ${{ matrix.os.emoji }} ${{ matrix.arch.emoji }} ${{ matrix.configuration.name }} - ${{ matrix.python.name }}
runs-on: ${{ matrix.os.runs-on }}
timeout-minutes: ${{ matrix.configuration.job_timeout }}
strategy:
fail-fast: false
matrix:
configuration: ${{ fromJson(inputs.configuration) }}
os:
- emoji: ${{ inputs.emoji }}
- emoji: ${{ inputs.os_emoji }}
matrix: ${{ inputs.matrix }}
name: ${{ inputs.name }}
file_name: ${{ inputs.file_name }}
runs-on: ${{ inputs.runs-on }}
arch:
- matrix: ${{ inputs.arch }}
emoji: ${{ inputs.arch-emoji }}
python:
- name: "3.8"
file_name: "3.8"
action: "3.8"
apt: "3.8"
install_sh: "3.8"
matrix: "3.8"
exclude_from:
limited: True
main: True
- name: "3.9"
file_name: "3.9"
action: "3.9"
Expand Down Expand Up @@ -109,10 +104,6 @@ jobs:
limited: True
main: True
exclude:
- os:
matrix: macos
python:
matrix: "3.8"
- arch:
matrix: arm
configuration:
Expand Down
14 changes: 9 additions & 5 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ jobs:
uses: ./.github/workflows/test-single.yml
needs: configure
with:
emoji: 🍎
os-emoji: 🍎
matrix: macos-intel
name: macOS Intel
file_name: macos
Expand All @@ -122,12 +122,13 @@ jobs:
matrix_mode: ${{ needs.configure.outputs.matrix_mode }}
runs-on: macos-13
arch: intel
arch-emoji: 🌀
macos-arm:
if: github.event_name != 'workflow_dispatch' || inputs.run-macos-arm
uses: ./.github/workflows/test-single.yml
needs: configure
with:
emoji: 🍎
os-emoji: 🍎
matrix: macos
name: macOS ARM
file_name: macos-arm
Expand All @@ -136,13 +137,14 @@ jobs:
matrix_mode: ${{ needs.configure.outputs.matrix_mode }}
runs-on: macos-14
arch: arm
arch-emoji: 💪
collect-junit: false
ubuntu:
if: github.event_name != 'workflow_dispatch' || inputs.run-linux
uses: ./.github/workflows/test-single.yml
needs: configure
with:
emoji: 🐧
os-emoji: 🐧
matrix: ubuntu
name: Ubuntu
file_name: ubuntu
Expand All @@ -151,13 +153,14 @@ jobs:
matrix_mode: ${{ needs.configure.outputs.matrix_mode }}
runs-on: ubuntu-latest
arch: intel
arch-emoji: 🌀
collect-junit: false
windows:
if: github.event_name != 'workflow_dispatch' || inputs.run-windows
uses: ./.github/workflows/test-single.yml
needs: configure
with:
emoji: 🪟
os-emoji: 🪟
matrix: windows
name: Windows
file_name: windows
Expand All @@ -166,6 +169,7 @@ jobs:
matrix_mode: ${{ needs.configure.outputs.matrix_mode }}
runs-on: windows-latest
arch: intel
arch-emoji: 🌀
collect-junit: false

coverage:
Expand Down Expand Up @@ -292,7 +296,7 @@ jobs:
- name: Coverage report (diff)
if: (github.base_ref != '' || github.event.before != '') && always()
env:
compare-branch: ${{ github.base_ref == '' && steps.parent-commit.hash || format('origin/{0}', github.base_ref) }}
compare-branch: ${{ github.base_ref == '' && steps.parent-commit.outputs.hash || format('origin/{0}', github.base_ref) }}
run: |
set -o pipefail
diff-cover --config-file=.diffcover.toml --compare-branch=${{ env.compare-branch }} --fail-under=100 --html-report=coverage-reports/diff-cover.html --markdown-report=coverage-reports/diff-cover.md coverage-reports/coverage.xml | tee coverage-reports/diff-cover-stdout
Expand Down
21 changes: 6 additions & 15 deletions .github/workflows/upload-pypi-source.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ permissions:

jobs:
mypy:
name: ${{ matrix.os.emoji }} ${{ matrix.check.name }} - ${{ matrix.os.name }} ${{ matrix.arch.name }} ${{ matrix.python.major_dot_minor }}
name: ${{ matrix.os.emoji }} ${{ matrix.arch.emoji }} ${{ matrix.check.name }} - ${{ matrix.os.name }} ${{ matrix.python.major_dot_minor }}
runs-on: ${{ matrix.os.runs-on[matrix.arch.matrix] }}
timeout-minutes: 20
strategy:
Expand All @@ -52,11 +52,12 @@ jobs:
intel: windows-latest
arch:
- name: ARM64
emoji: 💪
matrix: arm
- name: Intel
emoji: 🌀
matrix: intel
python:
- major_dot_minor: "3.8"
- major_dot_minor: "3.9"
- major_dot_minor: "3.10"
- major_dot_minor: "3.11"
Expand All @@ -68,12 +69,6 @@ jobs:
python manage-mypy.py build-mypy-ini
mypy
exclude:
- os:
matrix: macos
arch:
matrix: arm
python:
major_dot_minor: "3.8"
- os:
matrix: windows
arch:
Expand Down Expand Up @@ -125,12 +120,8 @@ jobs:
python:
- major_dot_minor: "3.10"
check:
- name: black
command: black --check --diff .
- name: flake8
command: flake8 benchmarks build_scripts chia tools *.py
- name: pylint
command: pylint benchmarks build_scripts chia tools *.py
- name: ruff
command: ruff format --check --diff .
- name: generated protocol tests
command: |
python3 -m chia._tests.util.build_network_protocol_files
Expand Down Expand Up @@ -174,7 +165,7 @@ jobs:
- check
strategy:
matrix:
python-version: [3.8]
python-version: [3.10]
os: [ubuntu-latest]

steps:
Expand Down
5 changes: 0 additions & 5 deletions .isort.cfg

This file was deleted.

Loading

0 comments on commit bd639ec

Please sign in to comment.