You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Possible problem is that the source regexp from ua-regexes-lite specify the minor version as optional /Edge?\/(\d+)(\.(\d+)|)(\.(\d+)|)/. Not sure if this is necessary here as EdgeHTML before Chromium seems to always specify the windows build. Removing this optional statement would fix the issue.
Otherwise a fix on how browserslist-useragent-regexp compiles this source regexp with Edge versions is needed.
The text was updated successfully, but these errors were encountered:
Would you like to work on a fix?
Current and expected behavior
Current
Output regexp for
Edge < 117
matchesEdge/120.0.0
user agent.Expected
Output regexp to not match
Edge/120.0.0
Reproduction
NA
browserslist-useragent-regexp version
v4.1.0
Possible solution
Possible problem is that the source regexp from ua-regexes-lite specify the minor version as optional
/Edge?\/(\d+)(\.(\d+)|)(\.(\d+)|)/
. Not sure if this is necessary here as EdgeHTML before Chromium seems to always specify the windows build. Removing this optional statement would fix the issue.Otherwise a fix on how
browserslist-useragent-regexp
compiles this source regexp with Edge versions is needed.The text was updated successfully, but these errors were encountered: