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

Add limit suffixes #945

Merged
merged 1 commit into from
Nov 14, 2023
Merged

Add limit suffixes #945

merged 1 commit into from
Nov 14, 2023

Conversation

davepl
Copy link
Contributor

@davepl davepl commented Nov 14, 2023

Add limit suffixes

Allows the user to specify shortcuts like K, M, G, and T with the -l (limit) flag

  • I read the contribution guidelines in CONTRIBUTING.md.
  • I placed my solution in the correct solution folder.
  • I added a README.md with the right badge(s).
  • I added a Dockerfile that builds and runs my solution.
  • I selected drag-race as the target branch.
  • All code herein is licensed compatible with BSD-3.

@davepl davepl requested a review from rbergen November 14, 2023 03:47
Copy link
Contributor

@rbergen rbergen left a comment

Choose a reason for hiding this comment

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

LGTM!

I see you changed the curly bracket placement for a number of function blocks to ANSI, and then add a custom atoll() in K&R style. I was just curious if there is a specific reason for this?

@rbergen rbergen merged commit ed64030 into drag-race Nov 14, 2023
324 checks passed
@rbergen rbergen deleted the dave-args branch October 21, 2024 15:37
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