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

DfciPkg/UnitTests/pip-requirements.txt: Remove pytool-library; bump robot framework #105

Closed

Conversation

makubacki
Copy link
Member

@makubacki makubacki commented Nov 15, 2022

Description

  • Removes edk2-pytool-library since it has a very old version and
    the module can be picked up from (1) the main repo
    pip-requirements.txt or (2) the edk2-pytool-extensions dependency.

  • Updates robotframework pip module version since it also now very old.

  • Breaking change?

    • Will this change break pre-existing builds or functionality without action being taken?
      No - Should not be breaking for the reasons mentioned in the description.

How This Was Tested

Todo

Integration Instructions

N/A

Signed-off-by: Michael Kubacki [email protected]

…botframework

- Removes edk2-pytool-library since it has a very old version and
  the module can be picked up from (1) the main repo
  pip-requirements.txt or (2) the edk2-pytool-extensions dependency.

- Updates robotframework pip module version since it also now very
  old.

Signed-off-by: Michael Kubacki <[email protected]>
@makubacki
Copy link
Member Author

@spbrogan - I only found this section in the documentation that is related to this change - DfciPkg/UnitTests/DfciTests - Setting up the HOST system.

It only references that the pip-requirements.txt file should be installed but does not mention the contents. It does reference Python 3.9.4 but updating that is a bit unrelated to this change and probably fine to leave.

@makubacki makubacki marked this pull request as ready for review November 29, 2022 03:02
@makubacki
Copy link
Member Author

@mikeytdisco, this is something I found when going through the pip modules pulled down in our pipelines.

I don't have an environment set up to run the tests. Have you been running them recently? Could you try including this change the next time you run the robot tests?

@makubacki
Copy link
Member Author

Comment to bump as a reminder

@makubacki
Copy link
Member Author

Moved to issue in mu_feature_dfci - microsoft/mu_feature_dfci#46

@makubacki makubacki closed this Mar 4, 2023
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.

3 participants