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.

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

Instantiate a new accumulator InstallRequirement for combine_install_requirements output #1519

Merged
merged 11 commits into from May 31, 2022

Commits on May 25, 2022

  1. Partially revert "Fix a regression where multiple declarations of a d…

    …ependency"
    
    (commit d720fe6)
    
    This brings back copy_ireq_dependencies and combines its use with that of the
    proxies-for-naming. Editable and url_requirement ireqs are not cached by the
    Repository, so when dependencies are queried for them once, no dependencies
    will be found later for the copy made by combine_install_requirements.
    
    copy_ireq_dependencies links those dependencies at the time of copy.
    richafrank committed May 25, 2022
    Copy the full SHA
    828e2d6 View commit details
    Browse the repository at this point in the history
  2. Failing test for case of resolver combining extras after the first

    round of resolution.
    
    By this point, we've frozen the dependencies.
    richafrank committed May 25, 2022
    Copy the full SHA
    46ec202 View commit details
    Browse the repository at this point in the history
  3. Copy the full SHA
    34a2416 View commit details
    Browse the repository at this point in the history
  4. Integrate microcat49's attribute copying fix for lost local req info

    Fixes jazzband#1054
    
    Co-authored-by: Andrew <harta5039@gmail.com>
    2 people authored and richafrank committed May 25, 2022
    Copy the full SHA
    41b4d37 View commit details
    Browse the repository at this point in the history
  5. Copy the full SHA
    b11a5c1 View commit details
    Browse the repository at this point in the history
  6. re-resolve dependencies of combined install_requirement

    Zhuyi Xue authored and richafrank committed May 25, 2022
    Copy the full SHA
    a9e647a View commit details
    Browse the repository at this point in the history
  7. Make a new InstallRequirement instance instead of mutating a copy

    when combining instances
    richafrank committed May 25, 2022
    Copy the full SHA
    f35a7ba View commit details
    Browse the repository at this point in the history
  8. Removed unused parameter

    richafrank committed May 25, 2022
    Copy the full SHA
    351e126 View commit details
    Browse the repository at this point in the history
  9. No longer need the name proxy, now that we're not mutating InstallReq…

    …uirement
    
    instances in combine_install_requirements
    richafrank committed May 25, 2022
    Copy the full SHA
    b065118 View commit details
    Browse the repository at this point in the history
  10. Copy the full SHA
    d8b9135 View commit details
    Browse the repository at this point in the history

Commits on May 31, 2022

  1. Test the entire resolver output

    to ensure nothing else was emitted
    richafrank committed May 31, 2022
    Copy the full SHA
    97471ea View commit details
    Browse the repository at this point in the history