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.

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

macro: Handle sorting empty lists #4230

Closed
bjhargrave opened this issue Jul 15, 2020 · 1 comment · Fixed by #4216
Closed

macro: Handle sorting empty lists #4230

bjhargrave opened this issue Jul 15, 2020 · 1 comment · Fixed by #4216
Assignees
@bjhargrave
Labels
maint-candidate Issues or pull requests that are candidates for a maintenance release
Milestone
5.2

Comments

@bjhargrave
Copy link
Member

Instead of emitting an error message, we should handle empty lists in the sort macros. The uniq macro already handles empty lists.
@bjhargrave bjhargrave added this to the 5.2 milestone Jul 15, 2020
@bjhargrave bjhargrave self-assigned this Jul 15, 2020
@bjhargrave bjhargrave linked a pull request Jul 15, 2020 that will close this issue
Maven Test-Cases header #4216
Merged
@bjhargrave
Copy link
Member Author

Fixed by 0feb035

@bjhargrave bjhargrave added the maint-candidate Issues or pull requests that are candidates for a maintenance release label Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bjhargrave bjhargrave

Labels
maint-candidate Issues or pull requests that are candidates for a maintenance release
Projects
None yet
Milestone
5.2
Development

Successfully merging a pull request may close this issue.

Maven Test-Cases header
1 participant
@bjhargrave