Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

Add permissions v2 support #4

Closed
wants to merge 3 commits into from
Closed

Add permissions v2 support #4

wants to merge 3 commits into from

Conversation

GoogolGenius
Copy link
Contributor

This is not yet tested and still a work in progress
@thesadru
Copy link
Owner

thesadru commented Jun 6, 2022

Maybe should just add **kwargs there at this point in case I miss a feature again.

@GoogolGenius
Copy link
Contributor Author

Eh, then you lose type hints because you would annotate *args and **kwargs with something like typing.Any or whatever.

@thesadru
Copy link
Owner

thesadru commented Jun 6, 2022

Just in addition to the already known args.

@thesadru thesadru marked this pull request as ready for review August 14, 2022 08:28
@GoogolGenius
Copy link
Contributor Author

I think I messed up the commit history lol

@GoogolGenius
Copy link
Contributor Author

I have no idea why the tests are failing

thesadru added a commit that referenced this pull request Aug 29, 2022
thesadru added a commit that referenced this pull request Aug 29, 2022
@GoogolGenius GoogolGenius deleted the feature/permissions-v2 branch August 30, 2022 04:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants