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

ci: Replace whitelist_externals with allowlist_externals and removed whitelist_externals. #214

Closed
simkimsia opened this issue Apr 2, 2023 · 0 comments

Comments

@simkimsia
Copy link
Contributor

simkimsia commented Apr 2, 2023

tox replaced whitelist_externals with allowlist_externals and removed whitelist_externals.

This PR fixes the tox.ini so that it uses the new option.

See also: tox.wiki/en/3.18.1/changelog.html#v3-18-0-2020-07-23
and python-poetry/poetry#7341

simkimsia added a commit to simkimsia/django-pattern-library that referenced this issue Apr 2, 2023
In tox.ini:

- Replace whitelist_externals with allowlist_externals because whitelist_externals is deprecated
- Added isolated_build = True
- Remove install_command where it calls tox_install.sh
- Drop tool.poetry.dev-dependencies and switch to using tool.poetry.extras in order for tox to detect them

In ci.yml
- Because now use tool.poetry.extras so when call `poetry install` need to incldue `—extras ‘test’` as argument

The last 2 points about dropping dev-dependencies and using extras are from https://github.com/orgs/python-poetry/discussions/4307#discussioncomment-1041658

Fixes torchbox#212, torchbox#214, torchbox#215, torchbox#216, torchbox#218
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

No branches or pull requests

2 participants