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.

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

Pinning pyyaml in progress #6259

Merged
merged 3 commits into from
Aug 14, 2023
Merged

Pinning pyyaml in progress #6259

merged 3 commits into from
Aug 14, 2023

Conversation

Thingus
Copy link
Contributor

@Thingus Thingus commented Aug 9, 2023

Bumping pyyaml to 6.0.1 to get around yaml/pyyaml#736

@cypress
Copy link

cypress bot commented Aug 9, 2023

Passing run #20540 ↗︎

0 4 0 0 Flakiness 0

Details:

Pin _down_ pyyaml pending resolution of #6015
Project: FlowAuth Commit: 4ea700f254
Status: Passed Duration: 00:53 💡
Started: Aug 14, 2023 9:28 AM Ended: Aug 14, 2023 9:28 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Thingus and others added 2 commits August 10, 2023 13:27
Pyyaml < 6 is broken with new Cython 3, so repin pyyaml where possible to >=6
@greenape greenape force-pushed the bump_pyyaml branch 2 times, most recently from 61db817 to abf12db Compare August 10, 2023 13:45
@greenape greenape marked this pull request as ready for review August 14, 2023 09:46
@codecov
Copy link

codecov bot commented Aug 14, 2023

Codecov Report

Merging #6259 (4ea700f) into master (52a392f) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6259   +/-   ##
=======================================
  Coverage   93.24%   93.25%           
=======================================
  Files         278      278           
  Lines       10832    10832           
  Branches      895      895           
=======================================
+ Hits        10100    10101    +1     
  Misses        602      602           
+ Partials      130      129    -1     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@greenape greenape merged commit bce6c9b into master Aug 14, 2023
7 checks passed
@greenape greenape deleted the bump_pyyaml branch August 14, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants