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

Setting local variable for timeOut #517

Conversation

Lucasdfg07
Copy link

@Lucasdfg07 Lucasdfg07 commented Oct 4, 2020

Description

Motivation for this PR. Linking the related issue is enough:

Closes #509

Proposed Changes

Changes that this PR will introduce. Are there any breaking changes?

Testing

How can the reviewer assert that the modifications are working as intended? A sequence of commands is just fine:

make install
curl example.com

@rafaveira3 rafaveira3 self-requested a review October 5, 2020 13:00
@rafaveira3 rafaveira3 self-assigned this Oct 7, 2020
Copy link
Contributor

@rafaveira3 rafaveira3 left a comment

Choose a reason for hiding this comment

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

Hey, @Lucasdfg07! Thanks for contributing to huskyCI. I have just reviewed your PR but, unfortunately, it does not resolve the problem we described in the issue. You may need to change a few logic in the API as well.

We are having a great discussion in this other PR and I will be very happy to have you there too. Feel free to send more commits on this PR, that we will happily review it again. 🙃

@rafaveira3
Copy link
Contributor

Closing this one due to long inactivity. Feel free to reopen it if you are still working on this one, @Lucasdfg07! 😃

@rafaveira3 rafaveira3 closed this Nov 25, 2020
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.

Make SecurityTests' timeout configurable
2 participants