Skip to content

Update browserstack.config.ts - Fix variable name#2

Open
arjunjh1011 wants to merge 2 commits intobrowserstack:mainfrom
arjunjh1011:main
Open

Update browserstack.config.ts - Fix variable name#2
arjunjh1011 wants to merge 2 commits intobrowserstack:mainfrom
arjunjh1011:main

Conversation

@arjunjh1011
Copy link

The capabilities patch object was assigning the correct property name for browserVersion / browser_version

@arjunjh1011 arjunjh1011 requested a review from a team as a code owner August 21, 2023 21:34
@dennispreciado
Copy link

@arjunjh1011 I have made the changes and I have additional suggestions for this pr.

  1. Webstorm IDE is generating an error stating it does not recognize the caps.browser_version variable.
  2. I suggest we add browser_version: 'latest' under caps on line 13
const caps = {
  browser: 'chrome',
  browser_version: 'latest',
}
Screenshot 2023-08-22 at 11 00 30 AM

@arjunjh1011
Copy link
Author

@dennispreciado Done. Please approve it satisfied with the change.

@arjunjh1011
Copy link
Author

Dont have the merge rights. One of the Browserstack folk would need to help out here :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants