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

docs: Update README so the example works verbatim #779

Merged
merged 2 commits into from
Jan 2, 2025

Conversation

sole
Copy link
Contributor

@sole sole commented Jan 2, 2025

The import was missing curly braces, and it was importing the wrong thing which led to the example failing, as also described in #649 that was closed by the author without sending a PR.

I thought the library code might have changed and/or the README had got out of sync, but after looking at the source I realised what was happening.

This little change helps people get started with the lib. Hope it helps!

sole added 2 commits January 2, 2025 10:09
The import was missing curly braces, and it was importing the wrong thing which led to the example failing, as also described in appium#649 that was closed by the author without sending a PR.

I thought the library code might have changed and/or the README had got out of sync, but after looking at the source I realised what was happening.

This little change helps people get started with the lib. Hope it helps!
Update README.md so the example works
Copy link

linux-foundation-easycla bot commented Jan 2, 2025

CLA Signed


The committers listed above are authorized under a signed CLA.

@mykola-mokhnach
Copy link
Contributor

Thank you @sole
Please sign the CLA and update the PR title according to https://www.conventionalcommits.org

@sole sole changed the title Update README so the example works feat: Update README so the example works verbatim Jan 2, 2025
@sole sole changed the title feat: Update README so the example works verbatim docs: Update README so the example works verbatim Jan 2, 2025
@sole
Copy link
Contributor Author

sole commented Jan 2, 2025

Thank you - I wasn't sure if you needed the CLA for a doc change. I hope the new PR title is OK now, let me know what exactly to use otherwise because the feedback of the action is non-existent other than 'failing' :-)

@mykola-mokhnach mykola-mokhnach merged commit 5bdcba6 into appium:master Jan 2, 2025
9 of 10 checks passed
Copy link

github-actions bot commented Jan 5, 2025

🎉 This PR is included in version 12.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@KazuCocoa KazuCocoa added the size:XS contribution size: XS label Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released size:XS contribution size: XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants