-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CI: Add job to validate conda-forge meta.yaml #53027
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
Merged
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
f1f9dfd
CI: Add job to validate conda-forge meta.yaml
mroeschke d685ae0
Rename file, test from git
mroeschke 910c315
Fix typo
mroeschke 8307e18
Merge remote-tracking branch 'upstream/main' into ci/conda/meta
mroeschke 5f92a26
fix concurrency
mroeschke ba5cf16
rename to meta.yaml, test over python versions
mroeschke 2fc5554
Fix recipe, add python version to job name
mroeschke 6fefa26
Try using mambabuild verify directly
mroeschke c49b257
seperate test from build
mroeschke 0ce0df5
Add check, and save to specific output location
mroeschke 6ec3751
Skip verify step
mroeschke bca9b10
Check if it's in package
mroeschke 77a1716
Merge remote-tracking branch 'upstream/main' into ci/conda/meta
mroeschke 853ccea
Try specifying another folder
mroeschke 7223d10
Try local
mroeschke ef4c120
Give full path?
mroeschke 4186ef3
Check where the package is
mroeschke b5c30c6
Whats in noarch?
mroeschke f4d0eb1
check linux-64
mroeschke 9a7caec
Skip --test
mroeschke 5ab25ba
Merge remote-tracking branch 'upstream/main' into ci/conda/meta
mroeschke 2a8f931
only run during main and 2.0 checks
mroeschke 8b997e6
Merge remote-tracking branch 'upstream/main' into ci/conda/meta
mroeschke 693dd9b
update meta
mroeschke File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,93 @@ | ||
{% set version = "2.0.1" %} | ||
|
||
package: | ||
name: pandas | ||
version: {{ version }} | ||
|
||
source: | ||
git_url: ../.. | ||
|
||
build: | ||
number: 1 | ||
script: | ||
- export PYTHONUNBUFFERED=1 # [ppc64le] | ||
- {{ PYTHON }} -m pip install -vv --no-deps --ignore-installed . # [not unix] | ||
- {{ PYTHON }} -m pip install -vv --no-deps --ignore-installed . --global-option="build_ext" --global-option="-j4" --no-use-pep517 # [unix] | ||
skip: true # [py<39] | ||
|
||
requirements: | ||
build: | ||
- python # [build_platform != target_platform] | ||
- cross-python_{{ target_platform }} # [build_platform != target_platform] | ||
- cython # [build_platform != target_platform] | ||
- numpy # [build_platform != target_platform] | ||
- {{ compiler('c') }} | ||
- {{ compiler('cxx') }} | ||
host: | ||
- python | ||
- pip | ||
- setuptools >=61.0.0 | ||
- cython >=0.29.33,<3 | ||
- numpy >=1.21.6 # [py<311] | ||
- numpy >=1.23.2 # [py>=311] | ||
- versioneer | ||
- tomli # [py<311] | ||
run: | ||
- python | ||
- {{ pin_compatible('numpy') }} | ||
- python-dateutil >=2.8.2 | ||
- pytz >=2020.1 | ||
- python-tzdata >=2022.1 | ||
|
||
test: | ||
imports: | ||
- pandas | ||
commands: | ||
- pip check | ||
# Skip test suite on PyPy as it segfaults there | ||
# xref: https://github.com/conda-forge/pandas-feedstock/issues/148 | ||
# | ||
# Also skip `test_rolling_var_numerical_issues` on `ppc64le` as it is a known test failure. | ||
# xref: https://github.com/conda-forge/pandas-feedstock/issues/149 | ||
{% set markers = ["not clipboard", "not single_cpu", "not db", "not network", "not slow"] %} | ||
{% set markers = markers + ["not arm_slow"] %} # [aarch64 or ppc64le] | ||
{% set extra_args = ["-n=2 -m " + " and ".join(markers)] %} | ||
{% set tests_to_skip = "_not_a_real_test" %} | ||
{% set tests_to_skip = tests_to_skip + " or test_rolling_var_numerical_issues" %} # [ppc64le] | ||
{% set tests_to_skip = tests_to_skip + " or test_std_timedelta64_skipna_false" %} # [ppc64le] | ||
{% set tests_to_skip = tests_to_skip + " or test_value_counts_normalized[M8[ns]]" %} # [ppc64le] | ||
{% set tests_to_skip = tests_to_skip + " or test_to_datetime_format_YYYYMMDD_with_nat" %} # [ppc64le] | ||
{% set tests_to_skip = tests_to_skip + " or (TestReductions and test_median_2d)" %} # [ppc64le] | ||
{% set extra_args = extra_args + ["-k", "not (" + tests_to_skip + ")"] %} | ||
- python -c "import pandas; pandas.test(extra_args={{ extra_args }})" # [python_impl == "cpython"] | ||
requires: | ||
- pip | ||
- pytest >=7.0.0 | ||
- pytest-asyncio >=0.17.0 | ||
- pytest-xdist >=2.2.0 | ||
- pytest-cov | ||
- hypothesis >=6.46.1 | ||
- tomli # [py<311] | ||
|
||
about: | ||
home: http://pandas.pydata.org | ||
license: BSD-3-Clause | ||
license_file: LICENSE | ||
summary: Powerful data structures for data analysis, time series, and statistics | ||
doc_url: https://pandas.pydata.org/docs/ | ||
dev_url: https://github.com/pandas-dev/pandas | ||
|
||
extra: | ||
recipe-maintainers: | ||
- jreback | ||
- jorisvandenbossche | ||
- msarahan | ||
- ocefpaf | ||
- TomAugspurger | ||
- WillAyd | ||
- simonjayhawkins | ||
- mroeschke | ||
- datapythonista | ||
- phofl | ||
- lithomas1 | ||
- marcogorelli |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm unusure about the file name. Using the original/standard one makes sense. But at the same time, without more context inside
ci/
the file name is not very descriptive. Totally up to you, just a thought, but since we specify the file name when callingconda mambabuild
maybe naming thisci/conda-recipe.yml
would be better?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. I renamed it to
ci/conda-recipe.yml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to change this back to
meta.yaml
to be recognized byconda mambabuild