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

Touchup README.md and fix whitespace issues #243

Closed
wants to merge 1 commit into from
Closed

Touchup README.md and fix whitespace issues #243

wants to merge 1 commit into from

Conversation

TomSweeneyRedHat
Copy link
Member

Signed-off-by: TomSweeneyRedHat [email protected]

README.md had a lingering whitespace issues. This will (hopefully) fix that and touches up a few other nits.

@rhatdan
Copy link
Member

rhatdan commented Jan 18, 2018

LGTM

@baude
Copy link
Member

baude commented Jan 18, 2018

bot, retest this please

Makefile Outdated
@@ -1,5 +1,5 @@
GO ?= go
EPOCH_TEST_COMMIT ?= dd0d35d
EPOCH_TEST_COMMIT ?= 05317d8
Copy link
Member

Choose a reason for hiding this comment

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

It looks like there's trailing whitespace on this. Could that make gitvalidation angry?

Copy link
Member

Choose a reason for hiding this comment

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

did it pass gitvalidation locally ?

@TomSweeneyRedHat
Copy link
Member Author

bot, retest this please

@TomSweeneyRedHat
Copy link
Member Author

Just like #230, Error on Red Hat CI without any logs/errors. Looks like the bot doesn't like me, can someone kick it please?

@mheon
Copy link
Member

mheon commented Jan 18, 2018

bot, retest this please

@mheon
Copy link
Member

mheon commented Jan 18, 2018

Still seeing gitvalidation errors - whitespace errors in 0aa2459

@TomSweeneyRedHat
Copy link
Member Author

Finally all green test buttons. @mheon @baude @rhatdan sorry about the headaches, thanks for the help in fixing, and PTAL! This should fix up the test errors plaguing others.

@mheon
Copy link
Member

mheon commented Jan 19, 2018

Going to merge to clean up gitvalidation issues

@mheon
Copy link
Member

mheon commented Jan 19, 2018

@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 2ae079f has been approved by mheon

@rh-atomic-bot
Copy link
Collaborator

⚡ Test exempted: pull fully rebased and already tested.

@TomSweeneyRedHat TomSweeneyRedHat deleted the dev/tsweeney/readme2 branch January 19, 2018 19:43
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 27, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants