Skip to content

Commit 261bbf3

Browse files
authored
Correct Disable Pandas Testing for GHA Nightly Builds (#966)
* Remove Mixed Space/Tabs in GHA Daily Test YAML * Fix Files in `text_examples` For Both Windows And Linux Paths * Readdition of #on: [push] This line is not necessary but is convenient when debugging.
1 parent efee08d commit 261bbf3

File tree

2 files changed

+43
-43
lines changed

2 files changed

+43
-43
lines changed

.github/workflows/daily-test-build.yml

Lines changed: 42 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -11,24 +11,24 @@ jobs:
1111
test-wheels-on-azure:
1212
runs-on: ubuntu-latest
1313
steps:
14-
- name: Get current date
15-
id: date
16-
run: echo "::set-output name=date::$(date +'%a-%Y-%m-%d')"
17-
- name: Get libtiledb short SHA
18-
run: echo "LIBTILEDB_SHA=$(git ls-remote https://github.com/TileDB-Inc/TileDB HEAD | cut -c1-7)" >> $GITHUB_ENV
19-
- name: Create Test Branch for Azure Wheel Nightly Build
20-
uses: peterjgrainger/[email protected]
21-
env:
22-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
23-
with:
24-
branch: 'azure-wheel-test-${{ steps.date.outputs.date }}-against-${{ env.LIBTILEDB_SHA }}'
14+
- name: Get current date
15+
id: date
16+
run: echo "::set-output name=date::$(date +'%a-%Y-%m-%d')"
17+
- name: Get libtiledb short SHA
18+
run: echo "LIBTILEDB_SHA=$(git ls-remote https://github.com/TileDB-Inc/TileDB HEAD | cut -c1-7)" >> $GITHUB_ENV
19+
- name: Create Test Branch for Azure Wheel Nightly Build
20+
uses: peterjgrainger/[email protected]
21+
env:
22+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
23+
with:
24+
branch: "azure-wheel-test-${{ steps.date.outputs.date }}-against-${{ env.LIBTILEDB_SHA }}"
2525

2626
test:
2727
runs-on: ${{ matrix.os }}
2828
strategy:
2929
matrix:
3030
os: [ubuntu-latest, macos-10.15, windows-latest]
31-
uninstall_pandas: [true, false]
31+
uninstall_pandas: [true, false]
3232

3333
permissions:
3434
issues: write
@@ -37,46 +37,46 @@ jobs:
3737
MACOSX_DEPLOYMENT_TARGET: 10.14
3838

3939
steps:
40-
- name: Set up Python
41-
uses: actions/setup-python@v2
40+
- name: Set up Python
41+
uses: actions/setup-python@v2
4242

43-
- name: Print Python version
44-
run: |
45-
which python
46-
which pip
47-
python --version
43+
- name: Print Python version
44+
run: |
45+
which python
46+
which pip
47+
python --version
4848
49-
- name: Print env
50-
run: printenv
49+
- name: Print env
50+
run: printenv
5151

52-
- name: Checkout TileDB-Py `dev`
53-
uses: actions/checkout@v2
52+
- name: Checkout TileDB-Py `dev`
53+
uses: actions/checkout@v2
5454

55-
- name: Install dependencies
56-
run: python -m pip install --upgrade -r misc/requirements_ci.txt
55+
- name: Install dependencies
56+
run: python -m pip install --upgrade -r misc/requirements_ci.txt
5757

58-
- name: Test without pandas
59-
run: python -m pip uninstall pandas
60-
if: ${{ matrix.uninstall_pandas }}
58+
- name: Test without pandas
59+
run: python -m pip uninstall -y pandas
60+
if: ${{ matrix.uninstall_pandas }}
6161

62-
- name: Build TileDB-Py
63-
run: |
64-
python setup.py build_ext --inplace --werror
65-
python setup.py install
62+
- name: Build TileDB-Py
63+
run: |
64+
python setup.py build_ext --inplace --werror
65+
python setup.py install
6666
67-
- name: Test TileDB-Py
68-
run: pytest -vv
67+
- name: Test TileDB-Py
68+
run: pytest -vv
6969

7070
create_issue_on_fail:
7171
runs-on: ubuntu-latest
7272
needs: test
7373
if: failure() || cancelled()
7474
steps:
75-
- name: Checkout TileDB-Py `dev`
76-
uses: actions/checkout@v2
77-
- name: Create Issue if Build Fails
78-
uses: JasonEtco/create-an-issue@v2
79-
env:
80-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
81-
with:
82-
filename: .github/workflows/daily-test-build-issue-template.md
75+
- name: Checkout TileDB-Py `dev`
76+
uses: actions/checkout@v2
77+
- name: Create Issue if Build Fails
78+
uses: JasonEtco/create-an-issue@v2
79+
env:
80+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
81+
with:
82+
filename: .github/workflows/daily-test-build-issue-template.md

tiledb/tests/test_examples.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ def test_examples(self, path):
2828
# - in tmpdir so we don't pollute the source tree
2929
# - with exit status checking (should fail tests if example fails)
3030
requires_pd = [
31-
f"{self.PROJECT_DIR}/examples/{fn}.py"
31+
os.path.join(self.PROJECT_DIR, "examples", f"{fn}.py")
3232
for fn in ["incomplete_iteration", "parallel_csv_ingestion"]
3333
]
3434
if not has_pandas() and path in requires_pd:

0 commit comments

Comments
 (0)