Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix python requirement #2242

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Fix python requirement #2242

wants to merge 14 commits into from

Conversation

isuruf
Copy link
Member

@isuruf isuruf commented Feb 16, 2025

Checklist

  • Added a news entry
  • Regenerated schema JSON if schema altered (python conda_smithy/schema.py)

@isuruf isuruf requested a review from a team as a code owner February 16, 2025 20:23
@beckermr
Copy link
Member

This won't pass because conda-recipe-manager needs 3.11.

@beckermr
Copy link
Member

This error is really odd

Fatal Python error: none_dealloc: deallocating None
Python runtime state: finalizing (tstate=0x000000011e604290)

Current thread 0x00000001f9090f80 (most recent call first):
  Garbage-collecting
  <no Python frame>
/Users/runner/work/_temp/fb66d3c5-4e5f-4fd9-b733-9f24f1931680.sh: line 1:  6125 Abort trap: 6           pytest tests

It seems unrelated to anything in this PR.

@beckermr
Copy link
Member

pre-commit.ci autofix

beckermr
beckermr approved these changes Feb 16, 2025
View reviewed changes
Copy link
Member

@beckermr beckermr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm on the build matrix and code adjustments. We'll need to figure out the ci errors.

@beckermr
Copy link
Member

pre-commit.ci autofix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@beckermr beckermr beckermr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@isuruf @beckermr