Skip to content

Commit

Permalink
Use diffpy.utils GitHub CI workflows to address codecov 0% problem (#87)
Browse files Browse the repository at this point in the history
* Move tests folder to top

* Revert commit

* Add pytest-cov

* Use the latest diffpy.utils CI workflow
  • Loading branch information
bobleesj authored Sep 19, 2024
1 parent b5a0366 commit 544d742
Show file tree
Hide file tree
Showing 9 changed files with 57 additions and 219 deletions.
30 changes: 5 additions & 25 deletions .github/workflows/check-news-item.yml
Original file line number Diff line number Diff line change
@@ -1,32 +1,12 @@
name: Check News Item
name: Check for News

on:
pull_request_target:
branches:
- main

permissions:
pull-requests: write
contents: read

jobs:
build:
runs-on: ubuntu-latest
name: Check News item
steps:

# note: the checkout will pull code from the base branch. This step should not pull code from the merge commit
- uses: actions/checkout@v4

- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: '3.12'
cache: 'pip'
cache-dependency-path: 'pyproject.toml'
- run: pip install PyGithub
- run: python .github/workflows/check-news.py
env:
PR_NUMBER: "${{ github.event.number }}"
GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
GITHUB_REPOSITORY: "${{ env.GITHUB_REPOSITORY }}"
check-news-item:
uses: Billingegroup/release-scripts/.github/workflows/_check-news-item.yml@v0
with:
project: diffpy.snmf
61 changes: 0 additions & 61 deletions .github/workflows/check-news.py

This file was deleted.

59 changes: 0 additions & 59 deletions .github/workflows/docs.yml

This file was deleted.

55 changes: 0 additions & 55 deletions .github/workflows/main.yml

This file was deleted.

21 changes: 21 additions & 0 deletions .github/workflows/matrix-and-codecov-on-merge-to-main.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
name: CI

on:
push:
branches:
- main
release:
types:
- prereleased
- published
workflow_dispatch:

jobs:
CI:
uses: Billingegroup/release-scripts/.github/workflows/_matrix-and-codecov-on-merge-to-main.yml@v0
with:
project: diffpy.snmf
c_extension: false
headless: false
secrets:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
19 changes: 0 additions & 19 deletions .github/workflows/pre-commit.yml

This file was deleted.

14 changes: 14 additions & 0 deletions .github/workflows/publish-docs-on-release.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
name: Build and Deploy Docs

on:
release:
types:
- published
workflow_dispatch:

jobs:
publish-docs-on-release:
uses: Billingegroup/release-scripts/.github/workflows/_publish-docs-on-release.yml@v0
with:
project: diffpy.snmf
c_extension: false
16 changes: 16 additions & 0 deletions .github/workflows/tests-on-pr.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
name: Tests on PR

on:
push:
branches:
- main
pull_request:
workflow_dispatch:

jobs:
tests-on-pr:
uses: Billingegroup/release-scripts/.github/workflows/_tests-on-pr.yml@v0
with:
project: diffpy.snmf
c_extension: false
headless: false
1 change: 1 addition & 0 deletions requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@ pytest
codecov
coverage
pytest-env
pytest-cov

0 comments on commit 544d742

Please sign in to comment.