Skip to content

Commit 73386fe

Browse files
authored
Merge pull request #48 from TomerFi/fixed-classifiers
Fixed classifiers
2 parents a0ebfa0 + 333aec9 commit 73386fe

File tree

8 files changed

+199
-56
lines changed

8 files changed

+199
-56
lines changed

.dependabot/config.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ update_configs:
1010
default_assignees:
1111
- "TomerFi"
1212
default_labels:
13-
- "dependencies"
13+
- "dependabot"
1414

1515
- package_manager: "python"
1616
directory: "/docs/"
@@ -21,4 +21,4 @@ update_configs:
2121
default_assignees:
2222
- "TomerFi"
2323
default_labels:
24-
- "dependencies"
24+
- "dependabot"

.github/PULL_REQUEST_TEMPLATE.md

+2-4
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
## Description
1+
# Description
2+
23
A clear and concise description of what was changed and why.
34

45
**Related issue (if any):** fixes #<issue number goes here>
5-
6-
## Checklist
7-
- [ ] I've followed the contributing guidelines.

.github/workflows/build.yml

+66-10
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,12 @@ on:
66
branch: dev
77

88
jobs:
9-
build:
9+
verify:
1010
runs-on: ubuntu-latest
1111
strategy:
1212
matrix:
1313
python: [3.8]
14-
name: Build project with Python ${{ matrix.python }}
14+
name: Verify project with Python ${{ matrix.python }}
1515
steps:
1616
- name: Source checkout
1717
uses: actions/checkout@v2
@@ -41,7 +41,7 @@ jobs:
4141
run: |
4242
pip install --upgrade pip poetry
4343
poetry config virtualenvs.create false --local
44-
poetry install -vv -E "checkers" -E "tests"
44+
poetry install -vv -E "checkers"
4545
4646
- name: Lint yaml files
4747
run: yamllint --config-file .yamllint --format colored --strict .
@@ -64,17 +64,73 @@ jobs:
6464
- name: Check static typing with mypy
6565
run: mypy --follow-imports silent --ignore-missing-imports src/ tests/ docs/
6666

67-
- name: Perform unit tests with coverage report
68-
run: pytest -vs --cov --cov-config=.coveragerc --cov-report=xml:coverage_report/coverage.xml
67+
test:
68+
runs-on: ubuntu-latest
69+
strategy:
70+
matrix:
71+
python: [3.6, 3.7, 3.8]
72+
name: Test project with Python ${{ matrix.python }}
73+
steps:
74+
- name: Source checkout
75+
uses: actions/checkout@v2
6976

70-
- name: Push coverage report to CodeCov
71-
uses: codecov/codecov-action@v1
77+
- name: Cache pip repository
78+
uses: actions/cache@v1
7279
with:
73-
file: ./coverage_report/coverage.xml
74-
fail_ci_if_error: true
80+
path: ~/.cache/pypoetry
81+
key: ${{ runner.os }}-pip-${{ hashFiles('**/poetry.lock') }}-${{ matrix.python }}
82+
restore-keys: |
83+
${{ runner.os }}-${{ matrix.python }}-pip-
84+
85+
- name: Cache poetry repository
86+
uses: actions/cache@v1
87+
with:
88+
path: ~/.cache/pypoetry
89+
key: ${{ runner.os }}-poetry-${{ hashFiles('**/poetry.lock') }}-${{ matrix.python }}
90+
restore-keys: |
91+
${{ runner.os }}-${{ matrix.python }}-poetry-
92+
93+
- name: Set up Python ${{ matrix.python }}
94+
uses: actions/setup-python@v1
95+
with:
96+
python-version: ${{ matrix.python }}
97+
98+
- name: Install dependencies
99+
run: |
100+
pip install --upgrade pip poetry
101+
poetry config virtualenvs.create false --local
102+
poetry install -vv -E "tests"
103+
104+
- name: Perform unit tests
105+
if: matrix.python != 3.8
106+
run: pytest -vs
107+
108+
- name: Perform unit tests with coverage report
109+
if: matrix.python == 3.8
110+
run: pytest -vs --cov --cov-config=.coveragerc --cov-report=xml:coverage.xml
75111

76112
- name: Upload coverage report as artifact
113+
if: matrix.python == 3.8
77114
uses: actions/upload-artifact@v1
78115
with:
79116
name: coverage-report
80-
path: ./coverage_report/coverage.xml
117+
path: ./coverage.xml
118+
119+
coverage:
120+
runs-on: ubuntu-latest
121+
name: Push coverage report to CodeCov
122+
needs: [verify, test]
123+
steps:
124+
- name: Source checkout
125+
uses: actions/checkout@v2
126+
127+
- name: Download coverage report from artifacts
128+
uses: actions/download-artifact@v1
129+
with:
130+
name: coverage-report
131+
132+
- name: Push to CodeCov
133+
uses: codecov/codecov-action@v1
134+
with:
135+
file: coverage-report/coverage.xml
136+
fail_ci_if_error: true

.github/workflows/prerelease.yml

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ on:
99
jobs:
1010
pre-release:
1111
runs-on: ubuntu-latest
12+
name: Build pre-release and publish to GitHub
1213
strategy:
1314
matrix:
1415
python: [3.8]

.github/workflows/release.yml

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ on:
99
jobs:
1010
release:
1111
runs-on: ubuntu-latest
12+
name: Build pre-release and publish to PyPi
1213
strategy:
1314
matrix:
1415
python: [3.8]

.github/workflows/wip.yml

+1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ on: pull_request
66
jobs:
77
set-pr-status:
88
runs-on: ubuntu-latest
9+
name: Set WIP PRs as pending
910
env:
1011
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
1112
steps:

poetry.lock

+102-33
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)