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

Improve list templates and fix failing tests due to bugfixes #1277

Merged
merged 5 commits into from
Jan 4, 2017

Conversation

vanstee
Copy link
Member

@vanstee vanstee commented Dec 21, 2016

Most of the changes here revolve around making list commands print scannable output. The only exception is rule list as other commands require an id, but we need to show the entire rule since that's really the only identifiable information available.

Tests were also fixed to remove newlines and join tags and extra break tags in hipchat output which were each related to recent greenbar bugfixes.

@vanstee vanstee added the review label Dec 21, 2016
@vanstee vanstee force-pushed the vanstee/list-command-improvements branch from d380e3b to c339d8a Compare December 22, 2016 19:54
@vanstee vanstee force-pushed the vanstee/list-command-improvements branch from c00e53f to 78b697f Compare January 3, 2017 16:34
@@ -1,15 +1,24 @@
~each var=$results~
~$item.compatible~
Copy link
Member Author

Choose a reason for hiding this comment

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

@mpeck Wasn't sure what this did. Let me know if we need to keep it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Confirmed with @mpeck that this can be removed.

@vanstee vanstee force-pushed the vanstee/list-command-improvements branch from 54297d8 to 2bf520a Compare January 4, 2017 15:20
@vanstee vanstee merged commit ae2608a into master Jan 4, 2017
@vanstee vanstee deleted the vanstee/list-command-improvements branch January 4, 2017 15:36
@vanstee vanstee removed the review label Jan 4, 2017
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.

1 participant