Skip to content

Commit 12c86c9

Browse files
ci: skip databricks tests on fork PRs
1 parent 754224c commit 12c86c9

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,8 @@ jobs:
123123
python -m pip install -e .[test]
124124
- name: Run tests
125125
run: |
126-
# TODO: better way to disable all cloud backend tests?
127-
pytest pins -m 'not fs_rsc and not fs_s3 and not fs_gcs and not fs_abfs and not skip_on_github'
126+
# Skip all tests requiring credentials, which aren't available on fork PRs.
127+
pytest pins -m 'not fs_rsc and not fs_s3 and not fs_gcs and not fs_abfs and not fs_dbc and not skip_on_github'
128128
129129
130130
build-docs:

pyproject.toml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,10 @@ universal = 1
8787
[tool.pytest.ini_options]
8888
markers = [
8989
"fs_file: mark test to only run on local filesystem",
90+
# NOTE: if you add a new marker here for a backend that requires credentials,
91+
# be sure to exclude it from the test command in .github/workflows/ci.yml
92+
# in the "test-fork" job, to avoid failing tests where GitHub Actions
93+
# secrets aren't available.
9094
"fs_s3: mark test to only run on AWS S3 bucket filesystem",
9195
"fs_gcs: mark test to only run on Google Cloud Storage bucket filesystem",
9296
"fs_abfs: mark test to only run on Azure Datalake filesystem",

0 commit comments

Comments
 (0)