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

Fixes #16000 - Listing product content for an activation key does not expose state #6232

Merged
merged 1 commit into from Aug 9, 2016

Conversation

Projects
None yet
4 participants
@ranjan
Copy link
Member

commented Aug 5, 2016

No description provided.

@theforeman-bot

This comment has been minimized.

Copy link

commented Aug 5, 2016

There were the following issues with the commit message:

  • length of the first commit message line for 6eab179042ddc3ddff3c22a96fd07c4de625df7b exceeds 65 characters
  • commit message for 6eab179042ddc3ddff3c22a96fd07c4de625df7b is not wrapped at 72nd column

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

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


This message was auto-generated by Foreman's prprocessor

@akofink

This comment has been minimized.

Copy link
Member

commented Aug 9, 2016

LGTM - ACK @daviddavis could you merge please?

@daviddavis

This comment has been minimized.

Copy link
Member

commented Aug 9, 2016

Done. Thanks @akofink @ranjan

@daviddavis daviddavis merged commit 028bd01 into Katello:master Aug 9, 2016

1 check passed

default Job result: SUCCESS
Details

kgaikwad added a commit to kgaikwad/katello that referenced this pull request Aug 30, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.