Skip to content

CI: skip tests when only files in doc/web changes (github actions) #41310

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 24 commits into from
Jun 12, 2021
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
7eaca3f
CI: skip tests when only files in doc/web changes (github)
May 4, 2021
ed48829
Fixed yaml file indent
May 4, 2021
175eb94
Merge remote-tracking branch 'upstream/master' into CI-action-on-change
May 5, 2021
644989c
Revert "CI: skip tests when only files in doc/web changes (github)"
May 5, 2021
719a524
CI: Don't run lint if only doc changes (github worklow)
May 5, 2021
2fa1f9a
CI: Don't run lint if only doc changes (azure)
May 5, 2021
b0dfb08
Merge remote-tracking branch 'upstream/master' into CI-action-on-change
May 5, 2021
9c2fc06
Fix (ex|in)clude logic in azure templates
May 5, 2021
e459da3
Fixed small "typo"
May 5, 2021
e193ca1
Fix location of the "path" parameter
May 5, 2021
0c27427
Removed extra "trigger"
May 5, 2021
431a3a4
Merge remote-tracking branch 'upstream/master' into CI-action-on-change
May 5, 2021
8910b6a
Merge remote-tracking branch 'upstream/master' into CI-action-on-change
May 12, 2021
2bc89aa
Fix for comment
May 12, 2021
d5d531a
Fix for comment
May 12, 2021
0637ecb
Fix for comment
May 12, 2021
e283453
Fix for comment
May 12, 2021
3529f7d
Fix for comment
May 12, 2021
25ca63d
Fix for comment
May 12, 2021
ffd9d7e
Only ignore on "pull_request" not on "push"
May 12, 2021
0048e25
Merge remote-tracking branch 'upstream/master' into CI-action-on-change
Jun 3, 2021
4fed568
Addresing comments
Jun 3, 2021
75a44f3
Merge remote-tracking branch 'upstream/master' into CI-action-on-change
Jun 3, 2021
cae7f81
Merge remote-tracking branch 'upstream/master' into CI-action-on-change
Jun 12, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 18 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -99,14 +99,29 @@ jobs:
- name: Set up pandas
uses: ./.github/actions/setup

# Github workflows built-in path filters
# don't work on a level of individual jobs or steps.
# So, we use this.
- uses: dorny/paths-filter@v2
id: changes
with:
filters: |
doc:
- 'doc/**'
web:
- 'web/**'

- name: Build website
run: |
source activate pandas-dev
python web/pandas_web.py web/pandas --target-path=web/build
if: steps.changes.outputs.web == 'true'

- name: Build documentation
run: |
source activate pandas-dev
doc/make.py --warnings-are-errors | tee sphinx.log ; exit ${PIPESTATUS[0]}
if: steps.changes.outputs.doc == 'true'

# This can be removed when the ipython directive fails when there are errors,
# including the `tee sphinx.log` in te previous step (https://github.com/ipython/ipython/issues/11547)
Expand All @@ -123,14 +138,15 @@ jobs:

- name: Upload web
run: rsync -az --delete --exclude='pandas-docs' --exclude='docs' --exclude='Pandas_Cheat_Sheet*' web/build/ docs@${{ secrets.server_ip }}:/usr/share/nginx/pandas
if: github.event_name == 'push'
if: ${{ github.event_name == 'push' && steps.changes.outputs.web == 'true' }}

- name: Upload dev docs
run: rsync -az --delete doc/build/html/ docs@${{ secrets.server_ip }}:/usr/share/nginx/pandas/pandas-docs/dev
if: github.event_name == 'push'
if: ${{ github.event_name == 'push' && steps.changes.outputs.doc == 'true' }}

- name: Move docs into site directory
run: mv doc/build/html web/build/docs

- name: Save website as an artifact
uses: actions/upload-artifact@v2
with:
Expand Down