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

Inability to use a timestamp with milli/micro seconds #234

Open
antonioturdo opened this issue Jul 6, 2024 · 0 comments
Open

Inability to use a timestamp with milli/micro seconds #234

antonioturdo opened this issue Jul 6, 2024 · 0 comments

Comments

@antonioturdo
Copy link

In the comment of the formatTime method of the Client class it is argued that it is possible to use the result of microtime(true), which is a float. But in the only place where the method is used, in the message method of the same class, the value is cast to integer, making it effectively impossible to pass a timestamp with the precision of milli / micro seconds.

For reference, I saw that the cast was added in this refactoring.

I am going to open a PR to remove the cast to integer.

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

No branches or pull requests

1 participant