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 some packages*.json files after the WinAppSDK udpate #14056

Merged
merged 4 commits into from
Nov 1, 2024

Conversation

JesseCol
Copy link
Contributor

@JesseCol JesseCol commented Oct 31, 2024

Description

I did a WinAppSDK update recently, there are some packages*.json files that I missed when doing that update.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Why

Without this change, the "e2e-test-app-fabric" doesn't build successfully.

What

This change modifies some *.json files so the new version of WindowsAppSDK is the expected version.

I found and ran the script "NuGetRestoreForceEvaluateAllSolutions.ps1"

Testing

cd packages\e2e-test-app-fabric
yarn windows --no-launch --no-deploy

Microsoft Reviewers: Open in CodeFlow

@Yajur-Grover
Copy link
Contributor

Looks like SampleAppFabric is also on the old WinAppSDK:

@JesseCol
Copy link
Contributor Author

Looks like SampleAppFabric is also on the old WinAppSDK:

Thanks! I've now found and ran the script "NuGetRestoreForceEvaluateAllSolutions.ps1" which does what we need. I've verified that after this there are no references to the old version number (1.5.240227000).

@JesseCol JesseCol closed this Oct 31, 2024
@JesseCol JesseCol reopened this Oct 31, 2024
@JesseCol
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s), but failed to run 1 pipeline(s).

@JesseCol JesseCol marked this pull request as ready for review October 31, 2024 18:25
@JesseCol JesseCol requested review from a team as code owners October 31, 2024 18:25
@JesseCol
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s), but failed to run 1 pipeline(s).

Copy link
Contributor

@TatianaKapos TatianaKapos left a comment

Choose a reason for hiding this comment

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

LGTM! Do we also need to backport this to 0.76?

@JesseCol
Copy link
Contributor Author

JesseCol commented Nov 1, 2024

LGTM! Do we also need to backport this to 0.76?

Thanks for looking! Yes, looks like we should backport to 0.76 because the first part of the WinAppSDK update is there. Can you point me to instructions?

@JesseCol JesseCol merged commit 032b7f4 into microsoft:main Nov 1, 2024
59 checks passed
Yajur-Grover pushed a commit to Yajur-Grover/react-native-windows that referenced this pull request Nov 4, 2024
…t#14056)

* Update some packages*.json files after the WinAppSDK udpate

* Found and ran the NuGetRestoreForceEvaluateAllSolutions.ps1 script

* Change files
TatianaKapos added a commit that referenced this pull request Nov 7, 2024
…14078)

* Update some packages*.json files after the WinAppSDK udpate (#14056)

* Update some packages*.json files after the WinAppSDK udpate

* Found and ran the NuGetRestoreForceEvaluateAllSolutions.ps1 script

* Change files

* Change files

---------

Co-authored-by: JesseCol <[email protected]>
Co-authored-by: Tatiana Kapos <[email protected]>
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.

4 participants