Strip whitespaces from element text should be#1954
Open
emanlove wants to merge 13 commits intorobotframework:masterfrom
Open
Strip whitespaces from element text should be#1954emanlove wants to merge 13 commits intorobotframework:masterfrom
emanlove wants to merge 13 commits intorobotframework:masterfrom
Conversation
Trying to see if there is another way to descrip the matrix and run a select portion of the test matrix of selenium, python, browser ..
- Added continuie on error - switched browser on "previous" config to chrome - change setup chrome version to latest
- Switched over to select.yml for pull request and pushes - Renamed the select workflow - Added an older rf version configuration to the test matrix
yuriverweij
approved these changes
Jan 19, 2026
Member
There was a problem hiding this comment.
Could we not generate a expiry date in the future and use that instead of having to update this every year?
Member
Author
There was a problem hiding this comment.
That would be good although I tend to push this off every year till it is that time of year. One problem is expiry dates can no longer be longer than a year. So it is not like we can set it so far into the future that it will just never expire.
Member
There was a problem hiding this comment.
how about this?
*** Test Cases ***
Add Cookie With Python Calculation
${expiry} = Evaluate int((datetime.datetime.now() + datetime.timedelta(days=330)).timestamp()) modules=datetime
Add Cookie Cookie1 value1 expiry=${expiry}
Member
There was a problem hiding this comment.
We should create a new type for the locator, this is a lot of duplication.
from typing import TypeAlias, Union
from selenium.webdriver.remote.webelement import WebElement
Locator: TypeAlias = WebElement | str
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is @jannek76 pull request with some adjustments for the changed library.