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

Bugfix: Position is not an option but an argument to click() #4438

Closed
wants to merge 1 commit into from

Conversation

k9ert
Copy link

@k9ert k9ert commented Apr 13, 2022

In the click-documentation, position is listed as an argument, not a s an option. Unfortunately i can't judge whether one or the other works as this seems to be flaky in our case.
I just wanted to do a PR instead of an issue in order to be more constructive in the case that i'm right but maybe i'm wrong here for some reason?

In the click-documentation, `position` is listed as an argument, not a s an option. Unfortunately i can't judge whether one or the other works as this seems to be flaky in our case.
I just wanted to do a PR instead of an issue in order to be more constructive in the case that i'm right but maybe i'm wrong here for some reason?
@netlify
Copy link

netlify bot commented Apr 13, 2022

👷 Deploy request for cypress-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit f57a3ba

@vercel
Copy link

vercel bot commented Apr 13, 2022

@k9ert is attempting to deploy a commit to the Cypress Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Apr 13, 2022

CLA assistant check
All committers have signed the CLA.

@k9ert
Copy link
Author

k9ert commented Apr 19, 2022

The reason my test was flaky had nothing to do with this workaround. So the workaround did work for me.
Feel free to close this issue but maybe a clarification of this would be good in the docs.

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