Skip to content

Use StrOrPromise from typeddjango/django-stubs #331

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

Closed

Conversation

piotrszyma
Copy link
Contributor

@piotrszyma piotrszyma commented Jan 27, 2023

I have made things!

StrOrPromise updates after typeddjango/django-stubs#689

A continuation of #301 with tests fix.

Feel free to cherrypick these changes to original PR and close this one.

@piotrszyma piotrszyma changed the title feat: update for StrOrPromise in typeddjango/django-stubs Use StrOrPromise from typeddjango/django-stubs Jan 27, 2023
Copy link

@mschoettle mschoettle left a comment

Choose a reason for hiding this comment

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

LGTM!

@intgr intgr force-pushed the pszyma/str-or-promise branch from 7a6cc17 to c7cb58d Compare February 3, 2023 16:29
@intgr
Copy link
Contributor

intgr commented Feb 3, 2023

Thanks! There was a conflict due to sorting imports in #335, which I solved.

Feel free to cherrypick these changes to original PR and close this one.

Since most of the work was authored by @terencehonles, I'll go that route, but I'll add Co-authored-by: tag for you as well.

@intgr
Copy link
Contributor

intgr commented Feb 3, 2023

Merged in #335.

@intgr intgr closed this Feb 3, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants