Skip to content

feat: support Pipenv - Pipfile.lock#5404

Merged
jeremylong merged 4 commits intomainfrom
pipfileLoock
Feb 10, 2023
Merged

feat: support Pipenv - Pipfile.lock#5404
jeremylong merged 4 commits intomainfrom
pipfileLoock

Conversation

@jeremylong
Copy link
Copy Markdown
Collaborator

@jeremylong jeremylong commented Jan 31, 2023

Adds support for Pipfile.lock per #4995.

This currently uses the exact same enable/disable settings as the PipfileAnalyzer - and will supersede the PipfileAnalyzer if bother a Pipfile and Pipfile.lock exist.

Adds support for Pipefile.lock per #4995
@boring-cyborg boring-cyborg Bot added core changes to core tests test cases labels Jan 31, 2023
@boring-cyborg boring-cyborg Bot added ant changes to ant cli changes to the cli maven changes to the maven plugin utils changes to utils labels Feb 1, 2023
@jeremylong jeremylong added this to the 8.1.0 milestone Feb 1, 2023
@aikebah
Copy link
Copy Markdown
Collaborator

aikebah commented Feb 6, 2023

Looks like autocorrect messed up your PR message (pipefile / pipelock / bother)

Code looks good to me.

One remark: I think it would be good to add some remark in the javadoc of both analyzers that they operate in tandem on the same activation property (with the new analyzer taking over when a lock file is present), so that when revisiting them in the future it's hinted by the docs that if one changes the other is likely to need change as well.

@jeremylong jeremylong changed the title feat: support Pipenv - Pipefile.lock feat: support Pipenv - Pipfile.lock Feb 9, 2023
Copy link
Copy Markdown
Collaborator

@aikebah aikebah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jeremylong jeremylong merged commit 4730296 into main Feb 10, 2023
@jeremylong jeremylong deleted the pipfileLoock branch February 10, 2023 10:48
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ant changes to ant cli changes to the cli core changes to core maven changes to the maven plugin tests test cases utils changes to utils

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants