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.

Sign up for GitHub

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

Jump to bottom

Remove RequirementSet.require_hashes #7068

Merged
merged 3 commits into from
Oct 20, 2019
Merged

Remove RequirementSet.require_hashes #7068

merged 3 commits into from
Oct 20, 2019

Commits on Oct 13, 2019

  1. Parameterize require_hashes within Resolver 

    Removes the only dynamic state from Resolver, at the cost of passing
an extra variable to 2 methods.
    @chrahunt
    chrahunt committed Oct 13, 2019
    Configuration menu
    Copy the full SHA
    e41b5bc View commit details
    Browse the repository at this point in the history

  2. Pass require_hashes directly to Resolver 

    This removes some of the dependence of the Resolver on our specific
RequirementSet implementation.
    @chrahunt
    chrahunt committed Oct 13, 2019
    Configuration menu
    Copy the full SHA
    bbc29f0 View commit details
    Browse the repository at this point in the history

  3. Remove unused RequirementSet.require_hashes

    @chrahunt
    chrahunt committed Oct 13, 2019
    Configuration menu
    Copy the full SHA
    b8fb97a View commit details
    Browse the repository at this point in the history