-
Notifications
You must be signed in to change notification settings - Fork 14
add helptext to read #242
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
base: main
Are you sure you want to change the base?
add helptext to read #242
Conversation
|
Hello My last pull request became valid after I fixed similar errors: https://ci.appveyor.com/project/GDATACyberDefenseAG/robotframework-flaui/builds/53165754/job/8tyq0dbeoneu1cvu#L738 |
Hmm, thank you for the direction. However, I can't find any issues with the tests. What exactly do you mean? According to the message, there is only one check failing—does someone need to review it? Or should I automatically increase the version? I’m thinking that might be the reason the build is failing. Is that what you mean? |
|
Actuall implementation pylint said ************* Module src.FlaUILibrary.flaui.module.property Has to be fixxed asweel |
|
Pls adjust under CHANGELOG.md new keyword as upcoming change. |
Hello @petr-kus . The links had line numbers at the end. You had to look at the top of the page after clicking them. Nepitwin already did a copy paste in later messages in the PR. Also, please look at this: https://github.com/GDATASoftwareAG/robotframework-flaui?tab=readme-ov-file#releasing-a-new-version---frequent-tasks |
Test are not the issue here it's pylint ************* Module src.FlaUILibrary.flaui.module.property In module.property you have to fix the trailing whitespaces and the exception which is called to raise correctly. You can verify pylint locally on your machine with keen.bat pylint |
Then all looks fine Version.py has not to be adjusted here |
I added the ability to read the help text using the keyword Get Property From Element.
Previously, there was no way to check an element’s help text at all. (except path search)
I have validated and tested the functionality in my local environment.