Skip to content

Upgrade gauge ts and chrome version#2730

Merged
zabil merged 9 commits intomasterfrom
upgrade-gauge-ts
Jul 17, 2024
Merged

Upgrade gauge ts and chrome version#2730
zabil merged 9 commits intomasterfrom
upgrade-gauge-ts

Conversation

@zabil
Copy link
Member

@zabil zabil commented Jun 11, 2024

  • Bump gauge-ts plugin version
  • Move windows builds to node 20
  • Upgrade chrome version

Fixes #2732

@zabil zabil marked this pull request as draft July 1, 2024 12:24
@zabil zabil marked this pull request as ready for review July 2, 2024 12:44
@zabil zabil marked this pull request as draft July 11, 2024 08:45
zabil added 8 commits July 16, 2024 09:26
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Check if chrome update fixes windows testing issue

Refer
#2732 (comment)

Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
@zabil zabil force-pushed the upgrade-gauge-ts branch from 358f33a to a980406 Compare July 16, 2024 08:27
@zabil zabil marked this pull request as ready for review July 16, 2024 09:34
@zabil zabil changed the title Upgrade gauge ts version Upgrade gauge ts and chrome version Jul 16, 2024
@gaugebot
Copy link

gaugebot bot commented Jul 16, 2024

@zabil Thank you for contributing to taiko. Your pull request has been labeled as a release candidate 🎉🎉.

Merging this PR will trigger a release.

Please bump up the version as part of this PR.

Instructions to bump the version can found at CONTRIBUTING.md

If the CONTRIBUTING.md file does not exist or does not include instructions about bumping up the version, please looks previous commits in git history to see what changes need to be done.

Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
return;
}
if (elementsLength !== elements.length) {
if (elements && elementsLength !== elements.length) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding this to avoid undefined error on elements

<body>

<iframe src="./HTMLElements.html" width="1440" height="900">
<iframe src="HTMLElements.html" width="800" height="600">
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use a reasonably sized viewport

@zabil zabil requested a review from NivedhaSenthil July 16, 2024 09:51
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix typescript running issues on newer versions of node

Copy link
Member

@NivedhaSenthil NivedhaSenthil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great !!

@zabil zabil merged commit 7a4f1a8 into master Jul 17, 2024
@chadlwilson chadlwilson deleted the upgrade-gauge-ts branch February 14, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Browser processes do not terminate on Windows with Chrome(ium) version 126 and above

2 participants