Skip to content

Conversation

@Rushabhshroff
Copy link
Member

No description provided.

Eric Wang and others added 14 commits March 20, 2022 18:07
 Changes to be committed:
	new file:   .browserstack
	new file:   .gitignore
	modified:   README.md
	new file:   __pycache__/conftest.cpython-38-pytest-7.1.1.pyc
	new file:   config/capabilities1.json
	new file:   config/capabilities2.json
	new file:   config/capabilities_local.json
	new file:   config/capabilities_parallel.json
	new file:   conftest.py
	new file:   requirements.txt
	new file:   test_set.py
@shwetashetye
Copy link

@Ankit098 Can you review this repo for Pytest?

@francisf francisf requested a review from Ankit098 August 4, 2022 13:16
Copy link
Collaborator

@Ankit098 Ankit098 left a comment

Choose a reason for hiding this comment

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

Suggested some changes around removing single test, updating caps to W3C and minimising redundant code..

@Rushabhshroff Rushabhshroff requested a review from Ankit098 August 5, 2022 12:47
@Rushabhshroff Rushabhshroff requested a review from Ankit098 August 8, 2022 08:57
Ankit098
Ankit098 previously approved these changes Aug 8, 2022
@francisf francisf merged commit 60dd01f into browserstack:main Aug 9, 2022
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.

4 participants