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

Converting to GitHub Actions #2699

Closed
wants to merge 28 commits into from
Closed

Converting to GitHub Actions #2699

wants to merge 28 commits into from

Conversation

dansiegel
Copy link
Member

Description

Switching automated builds from Azure DevOps to GitHub Actions to keep everything Prism tied closer to the main Prism Repo.

@dansiegel dansiegel mentioned this pull request Jul 28, 2022
3 tasks
@anpin anpin mentioned this pull request Jul 31, 2022
3 tasks
@stale
Copy link

stale bot commented Nov 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@dansiegel
Copy link
Member Author

@jeromelaban when you get a chance if you could help take a look at why the Uno side is failing it would be greatly appreciated

@@ -23,7 +23,7 @@ Prism.Wpf helps you more easily design and build rich, flexible, and easy to mai
<Reference Include="System.Configuration" />
</ItemGroup>

<ItemGroup Condition=" $(TargetFramework.StartsWith('netcore')) OR $(TargetFramework.StartsWith('net5'))">
<ItemGroup Condition=" $(TargetFramework.StartsWith('netcore')) OR $(TargetFramework.StartsWith('net6'))">

Choose a reason for hiding this comment

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

@dansiegel just wondering if it would be more beneficial to use the symbols NETFRAMEWORK and NETCOREAPP to do this portion for future target framework changes to Prism WPF.

Implementation for this condition would be: $(DefineConstants.Contains('NETCOREAPP'))

@dansiegel dansiegel closed this Feb 6, 2023
@dansiegel dansiegel deleted the gh-actions branch May 25, 2023 13:33
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.

None yet

2 participants