Skip to content

Comments

release: bump version to 1.5.0#227

Merged
radoering merged 1 commit intopython-poetry:mainfrom
radoering:release/1.5.0
Aug 20, 2023
Merged

release: bump version to 1.5.0#227
radoering merged 1 commit intopython-poetry:mainfrom
radoering:release/1.5.0

Conversation

@radoering
Copy link
Member

Changed

  • Drop support for Python 3.7 (#189).
  • Improve warning when the lock file is not consistent with pyproject.toml (#215).

Fixed

  • Fix an issue where markers for dependencies required by an extra were not generated correctly (#209).

@dimbleby
Copy link
Contributor

dimbleby commented Aug 19, 2023

Recent updates to poetry-core are eventually going to cause this repository to want the following change:

diff --git a/tests/test_exporter.py b/tests/test_exporter.py
index 73cb03a..5ba17bf 100644
--- a/tests/test_exporter.py
+++ b/tests/test_exporter.py
@@ -447,7 +447,7 @@ def test_exporter_can_export_requirements_txt_with_nested_packages_and_markers(
         "b": Dependency.create_from_pep_508(f"b==4.5.6 ; {marker_py_windows}"),
         "c": Dependency.create_from_pep_508(f"c==7.8.9 ; {marker_py_win32}"),
         "d": Dependency.create_from_pep_508(
-            f"d==0.0.1 ; {marker_py_win32.union(marker_py_windows)}"
+            f"d==0.0.1 ; {marker_py_windows.union(marker_py_win32)}"
         ),
     }

The usual absurd circular dependencies and entangled CI pipelines will make it less painful if you get ahead of this: it should be safe to make that change now.

@radoering
Copy link
Member Author

Many thanks.

Secrus
Secrus previously approved these changes Aug 20, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
3.2% 3.2% Duplication

@radoering radoering merged commit 41bd035 into python-poetry:main Aug 20, 2023
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.

3 participants