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

Replace old shared project with real one - Avalonia.PlatformSupport #7347

Merged
merged 2 commits into from
Jan 30, 2022

Conversation

maxkatz6
Copy link
Member

What does the pull request do?

Removed old shared project and replaces with normal C# project.
Removed code duplication between platform backends.

Breaking changes

  • Binary breacking change (types moved to another assembly)
  • Namespace change breaking change (AssetsLoader impl moved to another namespace)

Note, new project might be merged into Avalonia project with this PR. Otherwise we can consider Numerge-ing some more dlls.

@maxkatz6 maxkatz6 requested review from kekekeks and grokys January 12, 2022 08:17
@maxkatz6 maxkatz6 force-pushed the platform-support-project branch from 32c0b96 to f451b7c Compare January 12, 2022 08:30
@rstm-sf rstm-sf mentioned this pull request Jan 14, 2022
3 tasks
@maxkatz6 maxkatz6 mentioned this pull request Jan 14, 2022
@maxkatz6 maxkatz6 force-pushed the platform-support-project branch 4 times, most recently from c617deb to 2b0aecd Compare January 22, 2022 01:48
@maxkatz6 maxkatz6 force-pushed the platform-support-project branch from 2b0aecd to 933ee3c Compare January 29, 2022 00:28
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 0.10.999-cibuild0018310-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

@maxkatz6 maxkatz6 force-pushed the platform-support-project branch from 933ee3c to 4d866b5 Compare January 29, 2022 02:15
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 0.10.999-cibuild0018312-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

@maxkatz6 maxkatz6 enabled auto-merge January 30, 2022 18:07
@maxkatz6 maxkatz6 merged commit a2698bd into master Jan 30, 2022
@maxkatz6 maxkatz6 deleted the platform-support-project branch January 30, 2022 18:25
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 0.10.999-cibuild0018329-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

#if NETCOREAPP
IsCoreClr = true,
#elif NETFRAMEWORK
IsDotNetFramework = false,

Choose a reason for hiding this comment

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

IsDotNetFramework = true ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks. Will fix it in another pr later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants