Skip to content

testing in two steps #65

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.

Sign up for GitHub

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

Open
vuillaut wants to merge 11 commits into
base: actions_env
Choose a base branch
from
Open

testing in two steps #65

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
4b00670
testing in two steps
vuillaut Jun 10, 2021
3ac3c1e
same for skhep
vuillaut Jun 10, 2021
ab21940
check if changes in slides to build them
vuillaut Jun 10, 2021
4898304
build pages only if there is a change in docs
vuillaut Jun 10, 2021
3337165
testing with a change in docs
vuillaut Jun 10, 2021
37ba9e5
test if changes in environment.yml
vuillaut Jun 10, 2021
1019e9a
test if meaningful change in environment.yml
vuillaut Jun 10, 2021
b58b1e8
using correct action
vuillaut Jun 10, 2021
f2dab7a
more test
vuillaut Jun 10, 2021
70a8f13
more test
vuillaut Jun 10, 2021
2946eb5
test change in env.yml
vuillaut Jun 10, 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
8 changes: 8 additions & 0 deletions .github/workflows/build-slides.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,15 @@ name: Build LaTeX Slides
on: [push, pull_request]

jobs:
test_slides_change:
runs-on: ubuntu-latest
uses: tj-actions/verify-changed-files@v6.2
with:
files: |
packaging/slides/*
Copy link
Contributor

@maxnoe maxnoe Jun 10, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that recursive? So does it still work when packaging/slides/images/* is changed?

Copy link
Contributor

@maxnoe maxnoe Jun 10, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And I just realize that git is completely missing

Copy link
Contributor Author

@vuillaut vuillaut Jun 10, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's what I am trying to test.
At the moment recursive or not, no action start 😆

testing/slides/*
build-slides:
if: steps.verify-changed-files.outputs.files_changed == 'true'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
Expand Down
7 changes: 7 additions & 0 deletions .github/workflows/conda-skhep.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,14 @@ on: [push, pull_request]


jobs:
test_env_change:
runs-on: ubuntu-latest
uses: tj-actions/verify-changed-files@v6.2
with:
files: |
scikit-hep/environment.yml
build:
if: steps.verify-changed-files.outputs.files_changed == 'true'
runs-on: ${{ matrix.os }}
strategy:
max-parallel: 5
Expand Down
9 changes: 7 additions & 2 deletions .github/workflows/gh_pages.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,14 @@ on:
- main # Set a branch to deploy

jobs:
test_pages_change:
runs-on: ubuntu-latest
uses: tj-actions/verify-changed-files@v6.2
with:
files: |
docs/*
deploy:
if: steps.verify-changed-files.outputs.files_changed == 'true'
runs-on: ubuntu-18.04
steps:
- uses: actions/checkout@v2
Expand All @@ -26,8 +33,6 @@ jobs:
echo "$PWD";
ls;



- name: Deploy
uses: peaceiris/actions-gh-pages@v3
with:
Expand Down
18 changes: 13 additions & 5 deletions .github/workflows/python-package-conda.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,20 @@ on: [push, pull_request]


jobs:
test_env_change:
runs-on: ubuntu-latest
outputs:
env: ${{ steps.filter.outputs.env }}
steps:
- uses: dorny/paths-filter@v2
id: file-changes
with:
filters: |
env:
- environment.yml
build:
needs: test_env_changes
if: ${{ needs.test_env_changes.outputs.env == 'true' }}
runs-on: ${{ matrix.os }}
strategy:
max-parallel: 5
Expand All @@ -15,10 +28,6 @@ jobs:
- uses: actions/checkout@v2
with:
submodules: recursive
- uses: tj-actions/verify-changed-files@v6.2
with:
files: |
environment.yml
- uses: conda-incubator/setup-miniconda@v2
with:
auto-update-conda: true
Expand All @@ -31,7 +40,6 @@ jobs:
conda info

- name: Install dependencies
if: steps.verify-changed-files.outputs.files_changed == 'true'
shell: bash -leo pipefail {0}
run: |
conda install mamba -n base -c conda-forge
Expand Down
2 changes: 0 additions & 2 deletions docs/content/posts/welcome.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,10 +98,8 @@ https://indico.in2p3.fr/event/20306/contributions/95829/
</embed>
</object>


## Recorded lecture


<iframe width="100%" height="500" src="https://www.youtube.com/embed/TUrJ_IudVx8" title="YouTube video player" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>


Expand Down
3 changes: 3 additions & 0 deletions environment.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,3 +30,6 @@ dependencies:
- wheel
- pip:
- build
- ctaplot