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

Mypy giving error when using emojize #250

Closed
cap-jmk opened this issue Jan 26, 2023 · 6 comments · Fixed by #248
Closed

Mypy giving error when using emojize #250

cap-jmk opened this issue Jan 26, 2023 · 6 comments · Fixed by #248

Comments

@cap-jmk
Copy link

cap-jmk commented Jan 26, 2023

When using emojize inside a typed function, I recieve the following error:

error: Call to untyped function "emojize" in typed context  [no-untyped-call]

Do you know how to handle that?

@cvzi
Copy link
Contributor

cvzi commented Jan 26, 2023

Are you using the latest version of the package? There was a bug in earlier version that was fixed in #239

@cap-jmk
Copy link
Author

cap-jmk commented Jan 27, 2023

Yes, I am using v2.2.0. We could try using lia to fix it? https://codeberg.org/cap_jmk/lia

pip install spawn-lia
lia heal emoji

May the force be with you ;)

@cvzi
Copy link
Contributor

cvzi commented Jan 27, 2023

I am not familiar with lia what does it do?

@cvzi
Copy link
Contributor

cvzi commented Jan 27, 2023

I found the problem, the .tar.gz package doesn't ship the type information at the moment, i.e. the *.pyi files aren't included.

This will be fixed by #248

If I install from that pull request via pip install --force-reinstall git+https://github.com/cvzi/emoji@wheel then mypy is happy.

@cap-jmk
Copy link
Author

cap-jmk commented Jan 29, 2023

I am not familiar with lia what does it do?

It helps with CI. I am making frequently a lot of mistakes recently, so I code a package that helps mitigating them. It includes
1.) Linting
2.) Automated formatting of various sorts
3.) Typechecking
4.) Git checking

And thank you for the fix :)

@cvzi
Copy link
Contributor

cvzi commented Jan 30, 2023

Thanks, I'll check it out 🙂 Looks interesting

@cap-jmk cap-jmk closed this as completed Jan 31, 2023
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 a pull request may close this issue.

2 participants