From d720cdf4bdb5f78723591fd20d44102a36c7a4da Mon Sep 17 00:00:00 2001 From: Francisco Arceo Date: Wed, 11 Dec 2024 15:03:01 -0500 Subject: [PATCH] chore: Skip tests on doc updates (#4834) * feat: Make entity value_type mandatory with deprecation warning - Add deprecation warning when value_type is not specified for an entity - Add test cases to verify deprecation warning behavior - Prepare for making value_type mandatory in next release Issue: feast-dev/feast#4670 Co-Authored-By: Francisco Javier Arceo Signed-off-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * style: Fix import sorting in entity files - Reorder imports according to PEP8 - Group standard library imports together - Fix ruff linting issues Co-Authored-By: Francisco Javier Arceo Signed-off-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * chore: Skip tests for community/docs/examples paths Co-Authored-By: Francisco Javier Arceo --------- Signed-off-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --- .github/workflows/pr_integration_tests.yml | 4 ++++ .github/workflows/pr_local_integration_tests.yml | 4 ++++ .github/workflows/smoke_tests.yml | 7 ++++++- .github/workflows/unit_tests.yml | 7 ++++++- 4 files changed, 20 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pr_integration_tests.yml b/.github/workflows/pr_integration_tests.yml index 5a1b483b39..923c0b0335 100644 --- a/.github/workflows/pr_integration_tests.yml +++ b/.github/workflows/pr_integration_tests.yml @@ -6,6 +6,10 @@ on: - opened - synchronize - labeled + paths-ignore: + - 'community/**' + - 'docs/**' + - 'examples/**' # concurrency is currently broken, see details https://github.com/actions/runner/issues/1532 #concurrency: diff --git a/.github/workflows/pr_local_integration_tests.yml b/.github/workflows/pr_local_integration_tests.yml index 8b2f8c13d2..e6a9e3e8bd 100644 --- a/.github/workflows/pr_local_integration_tests.yml +++ b/.github/workflows/pr_local_integration_tests.yml @@ -7,6 +7,10 @@ on: - opened - synchronize - labeled + paths-ignore: + - 'community/**' + - 'docs/**' + - 'examples/**' jobs: integration-test-python-local: diff --git a/.github/workflows/smoke_tests.yml b/.github/workflows/smoke_tests.yml index 774d58d22b..9a898dd4c5 100644 --- a/.github/workflows/smoke_tests.yml +++ b/.github/workflows/smoke_tests.yml @@ -1,6 +1,11 @@ name: smoke-tests -on: [pull_request] +on: + pull_request: + paths-ignore: + - 'community/**' + - 'docs/**' + - 'examples/**' jobs: unit-test-python: runs-on: ${{ matrix.os }} diff --git a/.github/workflows/unit_tests.yml b/.github/workflows/unit_tests.yml index a8ddd397e3..6f46d12963 100644 --- a/.github/workflows/unit_tests.yml +++ b/.github/workflows/unit_tests.yml @@ -1,6 +1,11 @@ name: unit-tests -on: [pull_request] +on: + pull_request: + paths-ignore: + - 'community/**' + - 'docs/**' + - 'examples/**' jobs: unit-test-python: runs-on: ${{ matrix.os }}