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

[Easy] Makefile & QueryParam.__repr__ #35

Merged
merged 1 commit into from
Nov 16, 2022
Merged

[Easy] Makefile & QueryParam.__repr__ #35

merged 1 commit into from
Nov 16, 2022

Conversation

bh2smith
Copy link
Collaborator

  1. Makefile makes everything easier
  2. The repr method makes logging queries human readable

@bh2smith bh2smith requested a review from a team November 15, 2022 19:22
@bh2smith bh2smith added the easy label Nov 15, 2022
@bh2smith bh2smith merged commit 05bfe69 into main Nov 16, 2022
@bh2smith bh2smith deleted the query-param-repr branch November 16, 2022 09:22
@github-actions github-actions bot locked and limited conversation to collaborators Nov 16, 2022
@gentrexha
Copy link

Hey this is cool. How do you make use of the makefile now that you've added it?

@bh2smith
Copy link
Collaborator Author

bh2smith commented Nov 16, 2022

Oh right sorry!

in you shell you just run

make KEYWORD

For any of the keywords in the make file (you should also expect to see auto completion in your shell).

For example, my favourite is make check

@gentrexha
Copy link

Oh right sorry!

in you shell you just run

make KEYWORD

For any of the keywords in the make file (you should also expect to see auto completion in your shell).

For example, my favourite is make check

This is pretty cool! I've never heard about this before.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants