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

Cleanup pipeline #1845

Merged
merged 4 commits into from
May 3, 2022
Merged

Cleanup pipeline #1845

merged 4 commits into from
May 3, 2022

Conversation

VladislavAntonyuk
Copy link
Contributor

@VladislavAntonyuk VladislavAntonyuk commented Apr 22, 2022

Description of Bug

Issues Fixed

Behavioral Changes

PR Checklist

  • Has a linked Issue, and the Issue has been approved
  • Has tests (if omitted, state reason in description)
  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard

@VladislavAntonyuk VladislavAntonyuk force-pushed the fix-pipeline branch 4 times, most recently from 5a81849 to f2bbc91 Compare April 22, 2022 10:38
@VladislavAntonyuk VladislavAntonyuk marked this pull request as draft April 25, 2022 12:37
@VladislavAntonyuk VladislavAntonyuk marked this pull request as ready for review April 25, 2022 19:27
@@ -175,10 +175,8 @@ void SetColor()

flyoutStyle.Setters.Add(new Windows.UI.Xaml.Setter(FlyoutPresenter.BackgroundProperty, Element.Color.ToWindowsColor()));

#if UWP_18362
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we already have min version 18362. no need to check

@@ -45,19 +45,7 @@
<UseWindowsForms>true</UseWindowsForms>
</PropertyGroup>
<PropertyGroup Condition=" $(TargetFramework.StartsWith('uap10.0')) ">
<DefineConstants>$(DefineConstants);UAP10_0</DefineConstants>
</PropertyGroup>
<PropertyGroup Condition="'$(TargetFramework)' == 'uap10.0.14393'">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unused target frameworks

@@ -62,15 +62,6 @@ jobs:
pool:
vmImage: windows-2019
steps:
- task: CmdLine@2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed some redundant steps as we already have a clean solution.
Also copy changes from MCT azure-pipeline: https://github.com/CommunityToolkit/Maui/pull/283/files

@VladislavAntonyuk VladislavAntonyuk changed the title Cleanup pipeline, fix uwp package Cleanup pipeline May 2, 2022
@TheCodeTraveler TheCodeTraveler requested a review from jfversluis May 2, 2022 19:19
@jfversluis jfversluis merged commit 3f4b4c6 into main May 3, 2022
@jfversluis jfversluis deleted the fix-pipeline branch May 3, 2022 06:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants