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 PyYAML Checks #145

Merged
merged 5 commits into from
Aug 9, 2023
Merged

Fix PyYAML Checks #145

merged 5 commits into from
Aug 9, 2023

Conversation

ThomasLaPiana
Copy link
Contributor

@ThomasLaPiana ThomasLaPiana commented Aug 9, 2023 •
edited
Loading

Closes #

Description Of Changes

Trying to fix some issues related to PyYAML and recent Cython changes

Potential hope here: yaml/pyyaml#736

Code Changes

  • add explicit Cython pins in the CI checks

Steps to Confirm

  • list any manual steps taken to confirm the changes

Pre-Merge Checklist

  • All CI Pipelines Succeeded
  • Update CHANGELOG.md

@ThomasLaPiana ThomasLaPiana self-assigned this Aug 9, 2023
@ThomasLaPiana ThomasLaPiana mentioned this pull request Aug 9, 2023
Merged
8 tasks
@ThomasLaPiana ThomasLaPiana marked this pull request as ready for review August 9, 2023 10:42
@ThomasLaPiana ThomasLaPiana merged commit 4b20019 into main Aug 9, 2023
30 checks passed
@ThomasLaPiana ThomasLaPiana deleted the ThomasLaPiana-fix-pyyaml-checks branch August 9, 2023 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TheAndrewJackson TheAndrewJackson TheAndrewJackson approved these changes

@NevilleS NevilleS Awaiting requested review from NevilleS

@pattisdr pattisdr Awaiting requested review from pattisdr

Assignees

@ThomasLaPiana ThomasLaPiana

@TheAndrewJackson TheAndrewJackson

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ThomasLaPiana @TheAndrewJackson