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 a :test to numbers. Deprecate the non-keyword arguments. #172

Merged
merged 1 commit into from
Sep 24, 2023

Conversation

okamsn
Copy link
Owner

@okamsn okamsn commented Sep 24, 2023

  • Modify numbers, numbers-down, and numbers-up.
  • Fix signalling an error when arguments which give directions conflict.
  • Add more tests.

These arguments were meant to be similar to the arguments of Pythons range
feature, but, depending on prior knowledge, would generally produce worse code.
Cases in which the direction of the iteration (up or down) is unknown can now be
handled by the new :test argument, which is more flexible than the non-keyword
arguments anyway.

See also issue #167.

@okamsn okamsn added bug Something isn't working simplification Making things simpler deprecation/obsoletion Marking things for eventual removal labels Sep 24, 2023
- Modify `numbers`, `numbers-down`, and `numbers-up`.
- Fix signalling an error when arguments which give directions conflict.
- Add more tests.
- Update docs, changelog, and readme.

These arguments were meant to be similar to the arguments of Pythons `range`
feature, but, depending on prior knowledge, would generally produce worse code.
Cases in which the direction of the iteration (up or down) is unknown can now be
handled by the new `:test` argument, which is more flexible than the non-keyword
arguments anyway.
@okamsn okamsn merged commit 795f95e into master Sep 24, 2023
5 checks passed
@okamsn okamsn deleted the numbers-func branch February 25, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working deprecation/obsoletion Marking things for eventual removal simplification Making things simpler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant