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

chore(deps): update script to enable cypress #415

Closed
wants to merge 1 commit into from
Closed

chore(deps): update script to enable cypress #415

wants to merge 1 commit into from

Conversation

jasinliu
Copy link

Without --port, cypress can not visit the default URL. This is confusing.
image

Copy link
Member

@cexbrayat cexbrayat left a comment

Choose a reason for hiding this comment

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

Hi @jasinliu

Thank you for the pull request.
--host is not necessary by default: this is probably because you have a network configuration (security) that prevents cypress from properly working.

I don't think we should include this by default, as it exposes the application on the host.

@jasinliu
Copy link
Author

Hi @jasinliu

Thank you for the pull request. --host is not necessary by default: this is probably because you have a network configuration (security) that prevents cypress from properly working.

I don't think we should include this by default, as it exposes the application on the host.

Thank you very much for your quick reply. I get it.

But --host did helps for me on Windows11. I'll look into the reasons in my free time.

@jasinliu jasinliu closed this Dec 28, 2023
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