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

fix: Remove unnecessary pip upgrades in testing #72

Merged
merged 2 commits into from
Nov 16, 2024

Conversation

rmccorm4
Copy link
Contributor

@rmccorm4 rmccorm4 commented Nov 16, 2024

  1. Resolves local vs system package errors around pip installation in Ubuntu 24.04 tests by removing the pip install --upgrade pip. The pip upgrade shouldn't be necessary for any installations in these tests.

Example error before this change:

$ pip install --upgrade pip
Requirement already satisfied: pip in /usr/lib/python3/dist-packages (24.0)
Collecting pip
  Using cached pip-24.3.1-py3-none-any.whl.metadata (3.7 kB)
Using cached pip-24.3.1-py3-none-any.whl (1.8 MB)
Installing collected packages: pip
  Attempting uninstall: pip
    Found existing installation: pip 24.0
ERROR: Cannot uninstall pip 24.0, RECORD file not found. Hint: The package was installed by debian.
  1. Replaces the now-deprecated unittest assertEquals with assertEqual to avoid the following error when running the tests: "class has no attribute assertEquals"

@rmccorm4 rmccorm4 requested review from krishung5 and mc-nv November 16, 2024 02:38
mc-nv
mc-nv previously approved these changes Nov 16, 2024
@mc-nv
Copy link
Contributor

mc-nv commented Nov 16, 2024

@rmccorm4 , suppose base branch should be r24.11 isn't?

@mc-nv mc-nv dismissed their stale review November 16, 2024 17:49

Possible that base branch change is required.

@rmccorm4 rmccorm4 changed the base branch from main to r24.11 November 16, 2024 17:54
@mc-nv mc-nv merged commit 569dffe into r24.11 Nov 16, 2024
3 checks passed
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