Skip to content

Commit 2ce0b7f

Browse files
authored
remove redundant build step (#40)
* add MANIFEST.in * Revert "add MANIFEST.in" This reverts commit ca03d07. * remove redundant build steps
1 parent fe4a200 commit 2ce0b7f

File tree

1 file changed

+8
-17
lines changed

1 file changed

+8
-17
lines changed

.github/workflows/release-package.yml

+8-17
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ jobs:
88
build:
99
name: Build distribution 📦
1010
runs-on: ubuntu-latest
11+
timeout-minutes: 30
1112

1213
steps:
1314
- uses: actions/checkout@v4
@@ -29,22 +30,11 @@ jobs:
2930
AUTHSIGNAL_API_SECRET: ${{ secrets.AUTHSIGNAL_API_SECRET }}
3031
run: poetry run python -m authsignal.client_tests
3132

32-
- name: Verify package structure
33+
- name: Build and store main distribution
3334
run: |
34-
pip install check-manifest
35-
check-manifest
36-
37-
- name: Build main distribution
38-
run: python3 -m build
39-
40-
- name: Test package installation
41-
run: |
42-
python -m venv test_env
43-
source test_env/bin/activate
44-
pip install dist/*.whl
45-
# Add a basic import test here
46-
deactivate
47-
35+
poetry build
36+
ls -la dist/
37+
4838
- name: Store main distribution packages
4939
uses: actions/upload-artifact@v4
5040
with:
@@ -60,10 +50,11 @@ jobs:
6050
export PYTHONPATH=${PYTHONPATH}:$(pwd)/authsignal
6151
echo "BASE_VERSION=$(python -c 'from version import VERSION; print(VERSION)')" >> $GITHUB_ENV
6252
63-
- name: Build TestPyPI distribution
53+
- name: Build and store TestPyPI distribution
6454
run: |
6555
poetry version "${BASE_VERSION}.dev$(date +%s)"
66-
python3 -m build
56+
poetry build
57+
ls -la dist/
6758
6859
- name: Store test distribution packages
6960
uses: actions/upload-artifact@v4

0 commit comments

Comments
 (0)