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

upgrade request dependency & mocha version #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

milesvdw
Copy link

@milesvdw milesvdw commented Dec 21, 2022

Resolves #9

@cangencer
Copy link
Owner

@milesvdw thanks for the PR! I have not touched this project in a while - looks like the travis CI integration doesn't work for some reason - do you have a recommendation on how to fix it and/or use something else to verify the PR

@milesvdw
Copy link
Author

milesvdw commented Feb 1, 2023

Hmmm not sure - I was able to run all the mocha tests successfully locally for the PR. I don't have any visibility into the Travis CI, but maybe a brief call together would help us get to the bottom of the issue? I actually sent you an email about this some weeks ago, but you can reach me at [email protected]. I'm more than happy to make myself available at some time which is convenient for your timezone, I often work odd hours anyway.

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.

[email protected] security vulnerability
2 participants