This repository was archived by the owner on Sep 5, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 7
bug: Tests are being installed as part of this package #90
Copy link
Copy link
Closed
Labels
🚨This issue needs some love.This issue needs some love.api: memcacheIssues related to the googleapis/python-memcache API.Issues related to the googleapis/python-memcache API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Description
Similarly to Workflows here: googleapis/python-workflows#83 this package installs tests into site-packages, which can break user tests if their tests directory shares the same name.
As I am not sure how you package the release, I don't think I can contribute a PR with a fix, but please look into this.
To reproduce:
install google-cloud-memcache
observe tests package in your venv's site-packages
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
🚨This issue needs some love.This issue needs some love.api: memcacheIssues related to the googleapis/python-memcache API.Issues related to the googleapis/python-memcache API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.