Skip to content

Resolve poetry failure on a secondary repo#1132

Merged
kuqin12 merged 31 commits intomicrosoft:feature/ffa_enablementfrom
kuqin12:install_env
May 8, 2025
Merged

Resolve poetry failure on a secondary repo#1132
kuqin12 merged 31 commits intomicrosoft:feature/ffa_enablementfrom
kuqin12:install_env

Conversation

@kuqin12
Copy link
Copy Markdown
Contributor

@kuqin12 kuqin12 commented Mar 25, 2025

Description

This change intends to setup poetry environments for the TFA build process.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

This change is tested locally.

Integration Instructions

N/A

@github-actions github-actions bot added the language:python Pull requests that update Python code label Mar 25, 2025
@kuqin12 kuqin12 changed the base branch from feature/ffa_enablement to main March 25, 2025 03:36
@kuqin12 kuqin12 changed the base branch from main to feature/ffa_enablement March 25, 2025 03:39
@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Mar 25, 2025
Comment thread Platforms/QemuSbsaPkg/PlatformBuild.py Outdated
@Raymond-MS
Copy link
Copy Markdown
Contributor

I don't feel super comfortable signing off on these changes as I don't understand them well enough. Would it be okay if you pulled in another reviewer to take a look as well? As far as I can tell the changes look okay but I don't have the experience in this to confirm there are no issues.

Copy link
Copy Markdown
Contributor

@Raymond-MS Raymond-MS left a comment

Choose a reason for hiding this comment

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

LGTM, but I would like another pair of eyes if possible. :)

@kuqin12 kuqin12 merged commit 6f60ca6 into microsoft:feature/ffa_enablement May 8, 2025
7 of 10 checks passed
@kuqin12 kuqin12 deleted the install_env branch May 8, 2025 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:non-functional Does not have a functional impact language:python Pull requests that update Python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants