Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Android] Convert android projects to package reference #4310

Merged
merged 4 commits into from Nov 6, 2018
Merged

Conversation

PureWeen
Copy link
Contributor

@PureWeen PureWeen commented Nov 5, 2018

Description of Change

Converting Android projects over to package reference. Because of the support libraries, Android has the most interlaced nuget dependencies so getting it converted over to Package Reference makes dealing with those dependencies much easier. I've recently been needing to iterate through support libraries to test out new v28 features and having everything package reference will make this much simpler

Platforms Affected

  • Android

Testing Procedure

As long as the UI Tests all pass this change should be fine

PR Checklist

  • Has automated tests
  • Rebased on top of the target branch at time of PR
  • Changes adhere to coding standard

@PureWeen PureWeen added p/Android t/housekeeping ♻︎ Internal only changes, won't be included in release notes labels Nov 5, 2018
Copy link
Contributor

@kingces95 kingces95 left a comment

Choose a reason for hiding this comment

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

Should we also follow up with PackageReferences for all the other platforms? I could see rolling this out one by one to make it more digestible but it'd be a shame if we end up using a mix of old and new nuget references strategies.

Anyway, looks like we're having some build issues.

@PureWeen
Copy link
Contributor Author

PureWeen commented Nov 5, 2018

@kingces95

I could see rolling this out one by one to make it more digestible

That was my plan. I just really want the android projects that depend on large sets of nugets upgraded to make it easier and I didn't want this PR to stall on other things.

@PureWeen PureWeen added the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Nov 5, 2018
@PureWeen PureWeen removed the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Nov 6, 2018
Copy link
Contributor

@kingces95 kingces95 left a comment

Choose a reason for hiding this comment

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

Cool!

@PureWeen PureWeen added this to In Review in v3.5.0 Nov 6, 2018
@rmarinho rmarinho merged commit d7c20b7 into 3.4.0 Nov 6, 2018
v3.5.0 automation moved this from In Review to Done Nov 6, 2018
@PureWeen PureWeen added the approved Has two approvals, no pending reviews, and no changes requested label Nov 6, 2018
@StephaneDelcroix StephaneDelcroix deleted the packageref branch November 9, 2018 11:47
@PureWeen PureWeen added this to the 3.4.0 milestone Nov 15, 2018
@samhouts samhouts added this to Done in v3.4.0 Nov 19, 2018
@samhouts samhouts removed this from Done in v3.5.0 Nov 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Has two approvals, no pending reviews, and no changes requested p/Android t/housekeeping ♻︎ Internal only changes, won't be included in release notes
Projects
No open projects
v3.4.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants