Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

Fixes #20921 - use String.chomp instead of slicing to remove newlines #527

Merged
merged 1 commit into from Sep 14, 2017
Merged

Fixes #20921 - use String.chomp instead of slicing to remove newlines #527

merged 1 commit into from Sep 14, 2017

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Sep 14, 2017

No description provided.

@theforeman-bot
Copy link

Issues: #20921

@theforeman-bot
Copy link

There were the following issues with the commit message:

  • length of the first commit message line for 92fba96 exceeds 65 characters

If you don't have a ticket number, please create an issue in Redmine.

More guidelines are available in Coding Standards or on the Foreman wiki.


This message was auto-generated by Foreman's prprocessor

Copy link
Contributor

@johnpmitsch johnpmitsch left a comment

Choose a reason for hiding this comment

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

irb(main):005:0> `hammer -u admin -p changeme$ capsule list | grep "changed34" | awk '{ print $1 }'`.chomp
=> "1"

Much better way of doing things, thanks! :)

@johnpmitsch
Copy link
Contributor

@evgeni just needs a rebase from the hammer_cmd PR

@evgeni
Copy link
Member Author

evgeni commented Sep 14, 2017

rebased

@johnpmitsch johnpmitsch merged commit f7db11e into Katello:master Sep 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants