Skip to content

Conversation

Ilham-Habibullin
Copy link
Contributor

No description provided.

@Ilham-Habibullin
Copy link
Contributor Author

@ffMathy

@Ilham-Habibullin
Copy link
Contributor Author

seems like test was not working in that pr already 92f7e60

@ffMathy
Copy link
Owner

ffMathy commented Nov 13, 2019

Is there a way we can make it work though? The PR was merged on purpose despite failing tests, so that I would remember to fix it next time.

@Ilham-Habibullin
Copy link
Contributor Author

How about logging WARN instead? There are some complicated typescript stuff like infern. Don't know how it works

@ffMathy
Copy link
Owner

ffMathy commented Nov 13, 2019

That's okay. We'll skip it for now.

@ffMathy ffMathy merged commit d7f962b into ffMathy:master Nov 13, 2019
@Ilham-Habibullin
Copy link
Contributor Author

Ilham-Habibullin commented Nov 13, 2019 via email

@ffMathy
Copy link
Owner

ffMathy commented Nov 13, 2019

@Ilham-Habibullin
Copy link
Contributor Author

Ilham-Habibullin commented Nov 13, 2019 via email

@ffMathy
Copy link
Owner

ffMathy commented Nov 13, 2019

No, thank YOU ;)

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.

2 participants