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

Add missing build test dependency #2577

Merged
merged 2 commits into from Dec 2, 2022
Merged

Conversation

ssbarnea
Copy link
Member

@ssbarnea ssbarnea commented Dec 1, 2022

Fixes: #2576

Thanks for contribution

Please, make sure you address all the checklists (for details on how see
development documentation)!

  • ran the linter to address style issues (tox -e fix_lint)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • updated/extended the documentation

@ssbarnea ssbarnea marked this pull request as ready for review December 1, 2022 20:44
pyproject.toml Outdated
@@ -44,6 +44,7 @@ optional-dependencies.docs = [
"towncrier>=22.8",
]
optional-dependencies.testing = [
"build>=0.9.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's make it build[virtualenv] and you can drop trailing .0-s

Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
@gaborbernat gaborbernat merged commit 8630be3 into tox-dev:main Dec 2, 2022
@mgorny
Copy link
Contributor

mgorny commented Dec 2, 2022

Thank you!

@jcristau
Copy link

jcristau commented Jan 3, 2023

@ssbarnea was there a particular reason for bumping the chardet dependency (and others) here?

@ssbarnea
Copy link
Member Author

ssbarnea commented Jan 3, 2023

@jcristau I do not remember and do not plan to spend my free time investigating it. Feel free to check their changelog and see if there are good or not reasons for the bump.

@jcristau
Copy link

jcristau commented Jan 3, 2023

I did, and can't see any.
Too bad, thanks anyway. I guess there was already an unexplained bump in 0100423 so I'll just pin tox to <4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

build test dependency missing
4 participants