Skip to content
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

Move keyboard selection to welcome #6513

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

jknphy
Copy link
Contributor

@jknphy jknphy commented Jan 10, 2019

#6492 was merged but there was some mess with git and my code was reverted accidentally. Sending again PR:

Move keyboard selection code to welcome test module to have all the tests related to that screen in one place.

@rwx788
Copy link
Member

rwx788 commented Jan 10, 2019

That's really strange. No changes to file are even logged...

@rwx788 rwx788 merged commit cf10b19 into os-autoinst:master Jan 10, 2019
@dzedro
Copy link
Contributor

dzedro commented Jan 10, 2019

No changes were made to the file, @jknphy was just missing commits

@rwx788
Copy link
Member

rwx788 commented Jan 10, 2019

@dzedro not sure what do you mean, there is a commit: 9893bc3 which doesn't exist anymore... Could you please elaborate a bit? Same happened here: #6486

@dzedro
Copy link
Contributor

dzedro commented Jan 10, 2019

@rwx788 sorry I was confused with closed commit, yes it is strange, I was checking also previous commits and there is no record

@jknphy
Copy link
Contributor Author

jknphy commented Jan 10, 2019

There was an accidental force-push which drop the commit with the changes. It is visible here although hard to find it.

@jknphy jknphy deleted the move_keyboard_sel_to_welcome branch January 22, 2019 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants