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

ResolvePackageFileConflicts performance enhancements #1805

Merged
merged 6 commits into from
Dec 14, 2017
Merged

ResolvePackageFileConflicts performance enhancements #1805

merged 6 commits into from
Dec 14, 2017

Commits on Dec 8, 2017

  1. Speed up ResolvePackageFileConflicts by avoiding reading files for As… 

    …semblyVersion.

Allow for packages to override other packages by default.
    @eerhardt
    eerhardt committed Dec 8, 2017
    1 Configuration menu
    Copy the full SHA
    dd07581 View commit details
    Browse the repository at this point in the history

  2. Move new ConflictResolution files to the new directory.

    @eerhardt
    eerhardt committed Dec 8, 2017
    Configuration menu
    Copy the full SHA
    8e818fd View commit details
    Browse the repository at this point in the history

  3. Lazy load the package override dictionary so when there are no confli… 

    …cts, we don't need to build it.
    @eerhardt
    eerhardt committed Dec 8, 2017
    Configuration menu
    Copy the full SHA
    ed7cf09 View commit details
    Browse the repository at this point in the history

  4. Remove unnecessary copy of PackageConflictOverrides.

    @eerhardt
    eerhardt committed Dec 8, 2017
    Configuration menu
    Copy the full SHA
    2b05809 View commit details
    Browse the repository at this point in the history

Commits on Dec 9, 2017

  1. Add unit tests for PackageOverrideResolver.

    @eerhardt
    eerhardt committed Dec 9, 2017
    Configuration menu
    Copy the full SHA
    28e3268 View commit details
    Browse the repository at this point in the history

Commits on Dec 13, 2017

  1. PR Feedback 

    - fix typo
- Add functional test to ensure the conflict resolution optimizations results in the same References.
- Adding that test found 3 more packages to add to the default list of package overrides.
    @eerhardt
    eerhardt committed Dec 13, 2017
    Configuration menu
    Copy the full SHA
    f6a3f88 View commit details
    Browse the repository at this point in the history