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

Bump WASDK and BuildTools #13687

Merged
merged 9 commits into from
Mar 20, 2023
Merged

Bump WASDK and BuildTools #13687

merged 9 commits into from
Mar 20, 2023

Conversation

mattleibow
Copy link
Member

@mattleibow mattleibow commented Mar 3, 2023

Description of Change

This PR updates the WASDK and also the BuildTools. We no longer need the BuildTools reference in the project as we can depend on it ourselves and the new buildTransitive feature will automatically add it to the apps.

Issues Fixed

Fixes an issue when users update the build of the WSDK, the BuildTools get a bad version and also need to be updated.

@Eilon Eilon added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Mar 6, 2023
@mattleibow
Copy link
Member Author

The build is green because CI has VS installed and there are some "known path lookups" happening to find the tools in the VS install.

@mattleibow mattleibow added the do-not-merge Don't merge this PR label Mar 7, 2023
@mattleibow mattleibow self-assigned this Mar 8, 2023
@mattleibow mattleibow marked this pull request as ready for review March 15, 2023 22:11
@@ -24,6 +24,7 @@
</ItemGroup>
<ItemGroup Condition="$(TargetFramework.Contains('-windows'))">
<PackageReference Include="Microsoft.WindowsAppSDK" />
<PackageReference Include="Microsoft.Windows.SDK.BuildTools" />
Copy link
Member Author

Choose a reason for hiding this comment

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

We can remove this line once WASDK updates to depend on the newer version - or we may just keep this so we control the base version.

@mattleibow mattleibow changed the title We no longer need to reference BuildTools Bumo WASDK and BuildTools Mar 17, 2023
@mattleibow mattleibow changed the title Bumo WASDK and BuildTools Bump WASDK and BuildTools Mar 17, 2023
@mattleibow mattleibow added dependencies Pull requests that update a dependency file and removed do-not-merge Don't merge this PR labels Mar 17, 2023
@mattleibow mattleibow enabled auto-merge (squash) March 17, 2023 12:28
@mattleibow mattleibow merged commit f5580d4 into main Mar 20, 2023
@mattleibow mattleibow deleted the dev/remove-buildtools branch March 20, 2023 13:26
@github-actions github-actions bot locked and limited conversation to collaborators Dec 12, 2023
@samhouts samhouts added the fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions dependencies Pull requests that update a dependency file fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants