Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Allow ConfigSet.add_config to receive parameterized generics for of_type. #3288

Merged
merged 2 commits into from
May 31, 2024

Conversation

ssbarnea
Copy link
Member

@ssbarnea ssbarnea commented May 29, 2024

Allow ConfigSet.add_config to receive parameterized generics for of_type.

Fixes: #3287

  • ran the linter to address style issues (tox -e fix)
  • 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 added the bug:minor does not affect many people or has no big impact label May 29, 2024
@ssbarnea ssbarnea changed the title Allow use of list[str] inside plugins Allow ConfigSet.add_config to receive parameterized generics for of_type. May 29, 2024
@ssbarnea ssbarnea force-pushed the fix/type branch 2 times, most recently from f20e12f to c60aab8 Compare May 29, 2024 12:40
@ssbarnea ssbarnea marked this pull request as ready for review May 29, 2024 12:46
@ssbarnea ssbarnea requested a review from gaborbernat as a code owner May 29, 2024 12:46
@ssbarnea ssbarnea requested a review from a team May 29, 2024 13:10
Copy link
Member

@gaborbernat gaborbernat left a comment

Choose a reason for hiding this comment

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

I cannot accept anything without test...

@ssbarnea
Copy link
Member Author

I cannot accept anything without test...

But of course! Note that I was not able to use pytest skip marks to avoid py38 because interpreter failed to load the test file itself. Still, using an if before the test definition worked fine.

@ssbarnea ssbarnea marked this pull request as ready for review May 31, 2024 16:38
@ssbarnea ssbarnea requested a review from gaborbernat May 31, 2024 16:38
@gaborbernat gaborbernat merged commit 7317225 into main May 31, 2024
25 checks passed
@gaborbernat gaborbernat deleted the fix/type branch May 31, 2024 23:56
seyidaniels pushed a commit to seyidaniels/tox that referenced this pull request Jun 10, 2024
…f_type`. (tox-dev#3288)

* Allow use of list[str] inside plugins

Fixes: tox-dev#3287

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bot:chronographer:provided bug:minor does not affect many people or has no big impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeError: isinstance() argument 2 cannot be a parameterized generic
2 participants