Skip to content
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

Update GitHub Actions: Build Triggers, upload-artifact #720

Merged
merged 9 commits into from
Jan 13, 2025
1 change: 1 addition & 0 deletions .github/workflows/build-doxygen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ on:
push:
branches:
- main
- dev
# To run the default repository branch weekly on sunday, uncomment the following 2 lines
#schedule:
#- cron: '0 0 * * 0'
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/call-build-pkgdown.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ on:
pull_request:
branches:
- main
- dev
jobs:
call-workflow:
uses: nmfs-fish-tools/ghactions4r/.github/workflows/build-pkgdown.yml@main
3 changes: 1 addition & 2 deletions .github/workflows/call-doc-and-style-r.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,11 @@
name: call-doc-and-style-r
# on specifies the build triggers. See more info at https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows
on:
# workflow_dispatch requires pushing a button to run the workflow manually. uncomment the following line to add:
workflow_dispatch:
# Runs the workflow on each push to the main or master branch:
push:
branches:
- main
- dev
jobs:
call-workflow:
uses: nmfs-fish-tools/ghactions4r/.github/workflows/doc-and-style-r.yml@main
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/call-update-pkgdown.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@
name: call-update-pkgdown
# on specifies the build triggers. See more info at https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows
on:
# this workflow runs on pushes to main or master or any time a new tag is pushed
# this workflow runs on pushes to main or any time a new tag is pushed
push:
branches: [main, master]
branches: [main]
tags: ['*']
kellijohnson-NOAA marked this conversation as resolved.
Show resolved Hide resolved
jobs:
call-workflow:
Expand Down
13 changes: 3 additions & 10 deletions .github/workflows/get-gtest-codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,11 @@ on:
push:
branches:
- main
paths-ignore:
- .devcontainer
- .github
- 'LICENSE'
- 'README.md'
- 'CONTRIBUTING.md'
- 'Rbuildignore'
- '.gitignore'
- 'man'
- dev
pull_request:
branches:
- main
- dev

jobs:
build:
Expand Down Expand Up @@ -63,7 +56,7 @@ jobs:
gcovr --exclude tests/ --cobertura coverage.xml

- name: save coverage report
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: coverage.xml
path: coverage.xml
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/run-clang-format.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ on:
push:
branches:
- main
- dev

jobs:
job:
Expand All @@ -24,11 +25,10 @@ jobs:
# We use Google style to format code.
steps:
- uses: actions/checkout@v4
- uses: DoozyX/clang-format-lint-action@v0.14
- uses: DoozyX/clang-format-lint-action@v0.18.1
with:
source: './inst/include ./src ./tests/gtest'
extensions: 'hpp, cpp'
clangFormatVersion: 13.0.0
style: Google
inplace: True

Expand Down
11 changes: 7 additions & 4 deletions .github/workflows/run-clang-tidy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,14 @@ name: run-clang-tidy

on:
workflow_dispatch:
# The default build trigger is to run the action on every push and pull request, for any branch
push:
# To run the default repository branch weekly on sunday, uncomment the following 2 lines
#schedule:
#- cron: '0 0 * * 0'
branches:
- main
- dev
pull_request:
branches:
- main
- dev

jobs:
job:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/run-googletest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
ctest --test-dir build --parallel 16

- name: Upload log when tests failed
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: failure()
with:
name: googletest-log-${{ matrix.os }}
Expand Down
Loading