Skip to content

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

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

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