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

Update dependencies in all projects and templates #697

Merged
merged 5 commits into from
Mar 27, 2020

Conversation

TimLariviere
Copy link
Member

This is essentially the groundwork for #682.

  • Updated Paket to support Android 10 so we can upgrade our templates to benefit from AndroidX packages

  • Simplified our dependencies files (Packages.target and paket.dependencies) by removing unneeded packages and unpinning some versions

  • Updated versions where possible (still targeting XF 4.4 here)

  • Segregated "internal" and "external" dependencies in Packages.target

I tried to put a clear distinction between "internal" and "external" dependencies.

Internal dependencies are NuGet packages the various projects, inside the Fabulous repository, use in order to compile and run but aren't showing as requirements in the users projects when taking a reference to one of our NuGet packages (e.g. FSharp.Core -- users should be free to use the version they prefer within reasons).

External dependencies are dependencies that will be required to the users when taking a reference to one of our NuGet packages. (e.g. Xamarin.Forms)

The main idea is that internal dependencies can target the latest version available, since it won't affect users.
External dependencies should always target the lowest compatible major version (e.g. Xamarin.Forms should be 4.4.0.991265 - lowest version in the 4.4.0 bracket - even if a newer 4.4.0.991864 is available) to not force users to target a potentially bugging new version when an older one works perfectly.

@TimLariviere TimLariviere merged commit 0146032 into fabulous-dev:master Mar 27, 2020
@TimLariviere TimLariviere deleted the dependencies-update branch March 27, 2020 18:37
@TimLariviere TimLariviere mentioned this pull request Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant