Avoid redundant Darglint checks #3758
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| --- | |
| name: ci-fast | |
| on: | |
| workflow_dispatch: | |
| workflow_call: | |
| push: | |
| branches: [main] | |
| paths-ignore: [docs/**, docker/**, '*', '!pyproject.toml', '**.md'] | |
| pull_request: | |
| types: [opened, synchronize, ready_for_review] | |
| paths-ignore: [docs/**, docker/**, '*', '!pyproject.toml', '**.md'] | |
| concurrency: | |
| # New commit on branch cancels running workflows of the same branch | |
| group: ${{ github.workflow }}-${{ github.ref }} | |
| cancel-in-progress: true | |
| jobs: | |
| docstring-check: | |
| if: github.event.pull_request.draft == false | |
| runs-on: arc-runner-set | |
| steps: | |
| - name: Checkout code | |
| uses: actions/[email protected] | |
| with: | |
| fetch-depth: 0 | |
| - name: Get changed files | |
| id: changed-files | |
| uses: tj-actions/changed-files@v45 | |
| with: | |
| files: | | |
| tests/harness/**.py | |
| src/zenml/**/**.py | |
| files_separator: ' ' | |
| fetch_depth: 0 | |
| - name: Debug | |
| env: | |
| ALL_CHANGED_FILES: ${{ steps.changed-files.outputs.all_changed_and_modified_files }} | |
| run: echo ${ALL_CHANGED_FILES} | |
| - name: Set up Python | |
| if: steps.changed-files.outputs.any_modified == 'true' | |
| uses: actions/[email protected] | |
| with: | |
| python-version: '3.10' | |
| - name: Install darglint using uv | |
| if: steps.changed-files.outputs.any_modified == 'true' | |
| run: | | |
| curl -LsSf https://astral.sh/uv/install.sh | sh | |
| source $HOME/.cargo/env | |
| uv pip install --system darglint | |
| - name: Check docstrings | |
| if: steps.changed-files.outputs.any_modified == 'true' | |
| env: | |
| ALL_CHANGED_FILES: ${{ steps.changed-files.outputs.all_changed_and_modified_files }} | |
| run: bash scripts/docstring.sh ${ALL_CHANGED_FILES} | |
| # sqlite-db-migration-testing-random: | |
| # runs-on: arc-runner-set | |
| # env: | |
| # ZENML_ANALYTICS_OPT_IN: false | |
| # ZENML_DEBUG: true | |
| # # if team member commented, not a draft, on a PR, using /fulltest | |
| # if: github.event.pull_request.draft == false || github.event_name == 'workflow_dispatch' | |
| # steps: | |
| # - name: Checkout code | |
| # uses: actions/[email protected] | |
| # with: | |
| # fetch-depth: 0 | |
| # - name: Set up Python 3.9 | |
| # uses: actions/[email protected] | |
| # with: | |
| # python-version: '3.9' | |
| # - name: Test migrations across versions | |
| # run: bash scripts/test-migrations.sh sqlite random | |
| # spellcheck: | |
| # if: github.event.pull_request.draft == false | |
| # runs-on: ubuntu-latest | |
| # steps: | |
| # - name: Checkout code | |
| # uses: actions/[email protected] | |
| # - name: Spelling checker | |
| # uses: crate-ci/[email protected] | |
| # with: | |
| # files: . | |
| # config: ./.typos.toml | |
| # api-docs-test: | |
| # if: github.event.pull_request.draft == false | |
| # runs-on: ubuntu-latest | |
| # steps: | |
| # - name: Checkout code | |
| # uses: actions/[email protected] | |
| # - name: Set up Python 3.11 | |
| # uses: actions/[email protected] | |
| # with: | |
| # python-version: '3.11' | |
| # - name: Test API docs buildable | |
| # run: bash scripts/generate-docs.sh -v DUMMY -c | |
| # update-templates-to-examples: | |
| # # this doesn't work on forked repositories (i.e. outside contributors) | |
| # # so we disable template updates for those PRs / branches | |
| # if: github.event.pull_request.head.repo.full_name == 'zenml-io/zenml' && github.event.pull_request.draft | |
| # == false | |
| # uses: ./.github/workflows/update-templates-to-examples.yml | |
| # with: | |
| # python-version: '3.8' | |
| # os: ubuntu-latest | |
| # secrets: inherit | |
| # custom-ubuntu-setup-and-unit-test: | |
| # if: github.event.pull_request.draft == false | |
| # strategy: | |
| # matrix: | |
| # os: [arc-runner-set] | |
| # python-version: ['3.10'] | |
| # fail-fast: false | |
| # uses: ./.github/workflows/unit-test.yml | |
| # with: | |
| # python-version: ${{ matrix.python-version }} | |
| # os: ${{ matrix.os }} | |
| # secrets: inherit | |
| # custom-arc-runner-set-integration-test: | |
| # if: github.event.pull_request.draft == false | |
| # strategy: | |
| # matrix: | |
| # os: [arc-runner-set] | |
| # python-version: ['3.10'] | |
| # test_environment: [default, docker-server-docker-orchestrator-mysql] | |
| # fail-fast: false | |
| # uses: ./.github/workflows/integration-test-fast.yml | |
| # with: | |
| # os: ${{ matrix.os }} | |
| # python-version: ${{ matrix.python-version }} | |
| # test_environment: ${{ matrix.test_environment }} | |
| # secrets: inherit |