Skip to content

Conversation

@dcwangmit01
Copy link
Contributor

No description provided.

@fstab
Copy link
Owner

fstab commented May 12, 2019

Thanks for the PR. I'm mostly offline until Wednesday and will have a look after that.

@dcwangmit01
Copy link
Contributor Author

Thanks @fstab

Force push was to fix some test issues. Tests are still failing in CI. Unable to reproduce CI test failures on my system.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.5%) to 67.578% when pulling c94501e on dcwangmit01:webhook into 7ae298d on fstab:master.

@fstab fstab merged commit ba8d15c into fstab:master May 16, 2019
@fstab
Copy link
Owner

fstab commented May 16, 2019

Great PR, thanks! The failing tests on Windows are not related to the PR.

@dcwangmit01
Copy link
Contributor Author

Thanks for merging this.

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