Skip to content

Commit

Permalink
Fix markers being included as None/null (#5788)
Browse files Browse the repository at this point in the history
* Fix markers being included as None/null

* Add news fragment.
  • Loading branch information
matteius authored Jul 11, 2023
1 parent 30067b4 commit 57406bb
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions news/5788.bugfix.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fix regression markers being included as None/null in requirements command.
4 changes: 3 additions & 1 deletion pipenv/routines/requirements.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,9 @@ def requirements_from_deps(deps, include_hashes=True, include_markers=True):
)
markers = (
"; {}".format(package_info["markers"])
if include_markers and "markers" in package_info
if include_markers
and "markers" in package_info
and package_info["markers"]
else ""
)
extras = (
Expand Down

0 comments on commit 57406bb

Please sign in to comment.