Add resolution for dependencies - #7
Merged
Merged
Conversation
TG1999
force-pushed
the
resolution
branch
2 times, most recently
from
June 13, 2022 16:14
db8742c to
ee35e70
Compare
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Especially support line of up to 100 chars Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* Change format_resolution() args to return only either a tree or a parent/children structure. List is not needed. * rename resolution() to get_resolved_dependencies() *Streamline imports Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* Change resuults returned by resolve() to be a tuple of (initial_requirements, resolved_dependencies) * Refine doc and doc strings * Use stderr for CLI messages Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* Make it data driven and test the whole result againsta JSON file * Add new test for a requirements files * Use regeneration of fixtures with the PYINSP_REGEN_TEST_FIXTURES=yes environment variable. Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* This help better understand what comes from where. * Also add missing ISC license Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* Add new CLI flag "--use-pypi-json-api" to use exclusively the PyPI fast but incorrect JSON API. * Remove pypi_simple_repo_in_repos() from 'python_inspector.resolution' This is not longer needed * Also remove dangerous globals from function and methods arguments: * PythonInputProvider.__init__() where repos was a mutable list * get_resolved_dependencies() where repos was a mutable list * Refactor methods that dealt with PyPI JSON vs. simple by splitting them in two specialized methods Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com