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 types for get_client_ip parameters #110

Merged
merged 1 commit into from
Feb 4, 2024
Merged

Conversation

federicobond
Copy link
Contributor

No description provided.

@un33k
Copy link
Owner

un33k commented Feb 4, 2024

@federicobond please kindly change the PR against the CI branch. Thx

@federicobond federicobond changed the base branch from master to ci February 4, 2024 18:33
@federicobond
Copy link
Contributor Author

Hi @un33k! Like this?

@coveralls
Copy link

Coverage Status

coverage: 86.885% (+0.4%) from 86.441%
when pulling 007559c on federicobond:types
into 410e179 on un33k:ci.

@un33k un33k merged commit 2430b9c into un33k:ci Feb 4, 2024
16 checks passed
@federicobond federicobond deleted the types branch February 4, 2024 18:49
@federicobond
Copy link
Contributor Author

Thank you!

un33k added a commit that referenced this pull request Feb 4, 2024
* Add types for get_client_ip parameters (#110)

---------

Co-authored-by: Federico Bond <[email protected]>
@un33k
Copy link
Owner

un33k commented Feb 4, 2024

It is pushed to pypi at 6.0.4. Thanks again.

@federicobond
Copy link
Contributor Author

@un33k could it be that the package does not contain the py.typed file when installed via pip? This is preventing mypy from recognizing the types.

@federicobond
Copy link
Contributor Author

Might need to adjust package_data like this in setup.py.

@un33k
Copy link
Owner

un33k commented Feb 5, 2024

I'll push up the PRs as they arrive. Thx

@federicobond
Copy link
Contributor Author

I'm sorry, I did not understand what you said.

@un33k
Copy link
Owner

un33k commented Feb 6, 2024

I meant: community enhancement PRs are welcomed whether it'd be for setup.py or anything else.

un33k added a commit that referenced this pull request Apr 11, 2024
* Add types for get_client_ip parameters (#110)
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

Successfully merging this pull request may close these issues.

3 participants