richer get/list options #10

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.

Sign up for GitHub

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

Jump to bottom

Merged

nicks merged 1 commit into main from nicks/get

Oct 22, 2020

Member

nicks commented Oct 22, 2020

Hello @nicks,

Please review the following commits I made in branch nicks/get:

864b413 (2020-10-22 11:06:16 -0400)
richer get/list options

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

nicks force-pushed the nicks/get branch 4 times, most recently from ef4c810 to c67fd55 Compare

October 22, 2020 15:19


          richer get/list options

769b7d5

nicks force-pushed the nicks/get branch from c67fd55 to 769b7d5 Compare

October 22, 2020 15:19

nicks merged commit 0d754e1 into main

nicks deleted the nicks/get branch

October 22, 2020 15:31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet