Skip to content

Commit

Permalink
CI Use check-sdist to check sdist rather than check-manifest (scikit-…
Browse files Browse the repository at this point in the history
  • Loading branch information
lesteve authored Apr 9, 2024
1 parent d1d1596 commit e27425f
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 6 deletions.
6 changes: 6 additions & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
.* export-ignore
asv_benchmarks export-ignore
azure-pipelines.yml export-ignore
benchmarks export-ignore
build_tools export-ignore
maint_tools export-ignore
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
name: "Check Manifest"
name: "Check sdist"

on:
schedule:
- cron: '0 0 * * *'

jobs:
check-manifest:
check-sdist:
# Don't run on forks
if: github.repository == 'scikit-learn/scikit-learn'

Expand All @@ -19,15 +19,15 @@ jobs:
# scipy and cython are required to build sdist
run: |
python -m pip install --upgrade pip
pip install check-manifest scipy cython
pip install check-sdist
- run: |
check-manifest -v
check-sdist --inject-junk
update-tracker:
uses: ./.github/workflows/update_tracking_issue.yml
if: ${{ always() }}
needs: [check-manifest]
needs: [check-sdist]
with:
job_status: ${{ needs.check-manifest.result }}
job_status: ${{ needs.check-sdist.result }}
secrets:
BOT_GITHUB_TOKEN: ${{ secrets.BOT_GITHUB_TOKEN }}
6 changes: 6 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -203,3 +203,9 @@ exclude= '''
| sklearn/metrics/_pairwise_distances_reduction/_radius_neighbors.pyx
)
'''

[tool.check-sdist]
# These settings should match .gitattributes
sdist-only = []
git-only = [".*", "asv_benchmarks", "azure-pipelines.yml", "benchmarks", "build_tools", "maint_tools"]
default-ignore = false

0 comments on commit e27425f

Please sign in to comment.