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

1288626: Does not report pool ids as serial numbers, ignore duplicates #1357

Merged
merged 1 commit into from
Dec 14, 2015

Conversation

cnsnyder
Copy link
Member

@cnsnyder cnsnyder commented Dec 7, 2015

No description provided.

@wottop wottop self-assigned this Dec 11, 2015
@@ -54,7 +54,8 @@
parse_baseurl_info, format_baseurl, is_valid_server_info, \
MissingCaCertException, get_client_versions, get_server_versions, \
restart_virt_who, get_terminal_width, print_error, \
ProductCertificateFilter, EntitlementCertificateFilter
ProductCertificateFilter, EntitlementCertificateFilter, \
unique_list_items
Copy link
Contributor

Choose a reason for hiding this comment

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

Put up a few lines ith the other method names. More readable.

@wottop
Copy link
Contributor

wottop commented Dec 14, 2015

Other than the minor comments above, ack.

@cnsnyder cnsnyder force-pushed the csnyder/1288626_Do_not_display_pool_ids_as_serials branch from 878be82 to 98a7cc7 Compare December 14, 2015 15:48
@cnsnyder
Copy link
Member Author

@wottop I have updated the PR according to your comments above.

@cnsnyder cnsnyder force-pushed the csnyder/1288626_Do_not_display_pool_ids_as_serials branch from 98a7cc7 to c8e5d1b Compare December 14, 2015 16:08
@wottop
Copy link
Contributor

wottop commented Dec 14, 2015

ack

wottop added a commit that referenced this pull request Dec 14, 2015
…y_pool_ids_as_serials

1288626: Does not report pool ids as serial numbers, ignore duplicates
@wottop wottop merged commit 6e6017d into master Dec 14, 2015
@wottop wottop deleted the csnyder/1288626_Do_not_display_pool_ids_as_serials branch December 14, 2015 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants