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

disable cafile_* tests #9089

Merged

Conversation

lucacasonato
Copy link
Member

Ref #9083

@lucacasonato lucacasonato force-pushed the debug_cafile_install_remote_module branch from 7cdb971 to 21eb722 Compare January 12, 2021 10:57
@lucacasonato lucacasonato changed the title debug cafile_install_remote_module disable cafile_* tests Jan 12, 2021
Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

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

LGTM

@lucacasonato
Copy link
Member Author

I am not going to merge this, since the tests are still flaky, and I am just progressively disabling all of our HTTPS tests. There is something wrong in the HTTPS server in the test util. We need to find it.

@lucacasonato lucacasonato merged commit 18b3150 into denoland:master Jan 13, 2021
@lucacasonato lucacasonato deleted the debug_cafile_install_remote_module branch January 13, 2021 15:48
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