Skip to content

Conversation

@bdurrani
Copy link
Contributor

@bdurrani bdurrani commented Nov 15, 2018

closes #739

@kblok
Copy link
Member

kblok commented Nov 15, 2018

@bdurrani though it doesn't affect the final result. Try to start every new feature branching from master, so we don't get 17 commits when we should've got only one.

Take a look at the CodeFactor error.

@bdurrani
Copy link
Contributor Author

@bdurrani though it doesn't affect the final result. Try to start every new feature branching from master, so we don't get 17 commits when we should've got only one.

Take a look at the CodeFactor error.

I'm not sure what I'm doing wrong here. I merge any changes in your repo into mine on the master branch before creating a new branch or pushing any new changes. I've been following these steps here.

still needs tests
@kblok
Copy link
Member

kblok commented Nov 17, 2018

@bdurrani you can do something like this

git checkout upstream/master -b some-feature --no-track

@bdurrani
Copy link
Contributor Author

@bdurrani you can do something like this

git checkout upstream/master -b some-feature --no-track

I'll give this a shot! I'm aiming to wrap up this PR by the end of the day. Thanks

@bdurrani bdurrani changed the title [WIP] Feature/wait for target Feature/wait for target Nov 18, 2018
@kblok kblok changed the title Feature/wait for target Browser.waitForTarget Nov 19, 2018
Copy link
Member

@kblok kblok left a comment

Choose a reason for hiding this comment

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

Almost there. Great job!

@bdurrani
Copy link
Contributor Author

bdurrani commented Nov 20, 2018

strange.. That failing test was passing previously and the only change i made was remove that one redundant line in the test.
All the tests seem to pass locally

@kblok
Copy link
Member

kblok commented Nov 20, 2018

@bdurrani it was a hiccup :)

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.

Browser.waitForTarget

2 participants