Skip to content

compose: small cleanups (rename funcs and files)#899

Merged
maiamcc merged 1 commit into
masterfrom
maiamcc/compose-cleanup
Jan 2, 2019
Merged

compose: small cleanups (rename funcs and files)#899
maiamcc merged 1 commit into
masterfrom
maiamcc/compose-cleanup

Conversation

@maiamcc

@maiamcc maiamcc commented Jan 2, 2019

Copy link
Copy Markdown
Contributor

Hello @jazzdan,

Please review the following commits I made in branch maiamcc/compose-cleanup:

c8ba932 (2019-01-02 11:34:06 -0500)
compose: small cleanups (rename funcs and files)

Code review reminders, by giving a LGTM you attest that:

  • Commits are adequately tested
  • Code is easy to understand and conforms to style guides
  • Incomplete code is marked with TODOs
  • Code is suitably instrumented with logging and metrics

@maiamcc
maiamcc requested a review from jazzdan January 2, 2019 16:34

@jazzdan jazzdan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@maiamcc
maiamcc merged commit 0c032a6 into master Jan 2, 2019
@maiamcc
maiamcc deleted the maiamcc/compose-cleanup branch January 2, 2019 19:51
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