Skip to content

Remove redundant package install warning checks#1348

Merged
edmorley merged 1 commit intomainfrom
rm-outdated-warnings
Aug 15, 2022
Merged

Remove redundant package install warning checks#1348
edmorley merged 1 commit intomainfrom
rm-outdated-warnings

Conversation

@edmorley
Copy link
Copy Markdown
Member

Since:

  • InsecurePlatformWarning only affects Python versions that are no longer available on the current stacks.
  • scipy can be installed, so the warning about it not being compatible is wrong.
  • six/distribute are no longer installed automatically, so those warnings are wrong.

GUS-W-11594475.

@edmorley edmorley self-assigned this Aug 15, 2022
@edmorley edmorley force-pushed the rm-outdated-warnings branch from a579eaf to d6f1ac3 Compare August 15, 2022 09:51
@edmorley edmorley marked this pull request as ready for review August 15, 2022 10:00
@edmorley edmorley requested a review from a team as a code owner August 15, 2022 10:00
Since:
* `InsecurePlatformWarning` only affects Python versions that are
   no longer available on the current stacks.
* `scipy` can be installed, so the warning about it not being compatible
   is wrong.
* `six`/`distribute` are no longer installed automatically, so those
   warnings are wrong.

GUS-W-11594475.
@edmorley edmorley force-pushed the rm-outdated-warnings branch from d6f1ac3 to 73ccac5 Compare August 15, 2022 11:24
@edmorley edmorley enabled auto-merge (squash) August 15, 2022 11:24
@edmorley edmorley merged commit 3f22f6b into main Aug 15, 2022
@edmorley edmorley deleted the rm-outdated-warnings branch August 15, 2022 11:26
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.

2 participants