Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Add pollTimeout to options #193

Merged
merged 4 commits into from
Feb 13, 2023

Conversation

m4hdyar
Copy link
Contributor

@m4hdyar m4hdyar commented Feb 9, 2023

Hi,

Added pollTimeout to options and also readme.
Regarding #192.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Co-authored-by: Christian Bromann <git@bromann.dev>
@christian-bromann
Copy link
Contributor

Seems like some tests still fail.

@m4hdyar
Copy link
Contributor Author

m4hdyar commented Feb 10, 2023

Seems like some tests still fail.

Yeah had forgotten to edit the tests also. I also separated the rest of the default values from the test cases.

@m4hdyar
Copy link
Contributor Author

m4hdyar commented Feb 13, 2023

@christian-bromann Can you please merge this pull request ? Should I do something else regarding this PR? Because I want to create another pull request in electron service as well regarding to this issue and I'm waiting for this pull request.

Copy link
Contributor

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

LGTM 👍

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.

None yet

2 participants