jmespath / jmespath/jmespath.js

[feature request] allow null for sort_by() as like as sort()

Open
#24 0 comments 6 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
847
Forks
101
PR merge metrics
No merged PRs in 30d

Description

It show that sort() is allow null
http://jmespath.org/specification.html#sort

However I found that sort_by() will throw exception if expression return null

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the JMESPath sort() specification linked in the issue and comparing the sort() and sort_by() entry points. Reproduce the exception when sort_by() receives an expression returning null, then verify that null is handled consistently with sort() without changing other sorting behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
data
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.