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

Alphabets are hard... #17136

Conversation

NickLaMuro
Copy link
Member

I "dunne" goofed and in #17130 put the websockets log prior to the logs lines for the ones starting with "v"... so shameful...

That said, I put 90% of the blame on @bdunne for not catching my mistake in the first place and merging just because I alphabetized some code for him (that is my story and I am sticking to it)

Links

Steps for Testing/QA

  1. Recite your alphabet
  2. When you get to the letter 'v', take note of what comes next
  3. Look at the code changes
  4. Nod your head in agreement
  5. ?????
  6. DO STEPS 1-4 AGAIN FOR GOOD MEASURE
  7. Directly press the merge button. Do NOT pass GO. Do NOT collect $200. (aka. no profit)

"Yes kids, the letter 'w' does indeed come after the letter 'v'.  Take
notes, because this will be on your exam and useful in your future
careers."
@NickLaMuro
Copy link
Member Author

NickLaMuro commented Mar 9, 2018

cc @bdunne @simaishi @gmcculloug ... anyone else who wants to mock an ridicule me to end your Friday.

@miq-bot
Copy link
Member

miq-bot commented Mar 9, 2018

Checked commit NickLaMuro@833f6b0 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@gmcculloug
Copy link
Member

@simaishi This PR goes along with PR #17130 for BZ https://bugzilla.redhat.com/show_bug.cgi?id=1553813 when back-porting.

@gmcculloug gmcculloug merged commit 37d1b17 into ManageIQ:master Mar 11, 2018
@simaishi
Copy link
Contributor

Combining into #17130 for backport.

@agrare agrare added this to the Sprint 81 Ending Mar 12, 2018 milestone Mar 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants