Skip to content

Conversation

@potapovDim
Copy link
Contributor

Motivation/Description of the PR

  • Fix inheritance

Type of change

  • [ x] 💅 Polish code

Checklist:

  • [ x] Lint checking (Run npm run lint)
  • [ x] Local tests are passed (Run npm test)

s-syam and others added 4 commits February 25, 2020 12:08
Earlier it seems like it was clicking the first element. Now it clicks the first visible element,
…on (codeceptjs#2228)

* add missing JSDoc parameter for seeInPopup method, update documentation

* refactor duplicated JSDoc of seeInSource method to mustache file
Edits update to new generator version, minor typos, clarifications.

Otherwise good overview docs!
@potapovDim
Copy link
Contributor Author

@DavertMik Hey, should i remove this PR?

@potapovDim potapovDim force-pushed the small_fixes branch 3 times, most recently from c8b13db to ce0c950 Compare March 17, 2020 18:38
* remove let, add const

* fix import error
Copy link
Contributor Author

@potapovDim potapovDim left a comment

Choose a reason for hiding this comment

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

done

@DavertMik
Copy link
Contributor

Codacy Here is an overview of what got changed by this pull request:

Clones added
============
- lib/helper/WebDriverIO.js  1
- lib/helper/WebDriver.js  3
         

Clones removed
==============
+ lib/command/run.js  -1
+ test/helper/Playwright_test.js  -1
+ test/runner/dry_run_test.js  -1
+ test/helper/Puppeteer_test.js  -1
         

See the complete overview on Codacy

@potapovDim potapovDim requested a review from Vorobeyko March 19, 2020 19:31
@DavertMik DavertMik changed the base branch from master to codeceptjs-v3.0 March 24, 2020 18:45
@DavertMik DavertMik merged commit dc86bc1 into codeceptjs:codeceptjs-v3.0 Mar 27, 2020
@potapovDim potapovDim deleted the small_fixes branch April 19, 2020 07:02
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.

8 participants