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

pyproject: use PyYAML 6.0.1 #1240

Merged

Conversation

jremmet
Copy link
Contributor

@jremmet jremmet commented Jul 20, 2023

With the release of Cython3 the build of PyYAML 5.4.1 is broken [1]. There seems to be no plan to fix 5.4.x branch [2]. Tox tests and a real world example runs fine with 6.0.1

[1] yaml/pyyaml#724
[2] yaml/pyyaml#728

master #1237

Description

Checklist

  • Documentation for the feature
  • Tests for the feature
  • The arguments and description in doc/configuration.rst have been updated
  • Add a section on how to use the feature to doc/usage.rst
  • Add a section on how to use the feature to doc/development.rst
  • PR has been tested
  • Man pages have been regenerated

With the release of Cython3 the build of PyYAML 5.4.1 is broken [1].
There seems to be no plan to fix 5.4.x branch [2].
Tox tests and a real world example runs fine with 6.0.1

[1] yaml/pyyaml#724
[2] yaml/pyyaml#728

Signed-off-by: Jan Remmet <j.remmet@phytec.de>
@codecov
Copy link

codecov bot commented Jul 20, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (ba15037) 63.2% compared to head (b41ab7a) 63.2%.

Additional details and impacted files
@@             Coverage Diff             @@
##           stable-23.0   #1240   +/-   ##
===========================================
  Coverage         63.2%   63.2%           
===========================================
  Files              156     156           
  Lines            11596   11596           
===========================================
  Hits              7334    7334           
  Misses            4262    4262           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Bastian-Krause Bastian-Krause merged commit b338f52 into labgrid-project:stable-23.0 Jul 20, 2023
10 checks passed
@jremmet jremmet deleted the pyyaml-6.0.1_stable-23.0 branch July 21, 2023 06:12
@Bastian-Krause Bastian-Krause added the dependencies Pull requests that update a dependency file label Jul 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file fix for stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants