fix: remove let support for now#57
Conversation
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
|
@springcomp WDYT ? Is it ok to remove |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #57 +/- ##
==========================================
- Coverage 80.01% 79.61% -0.40%
==========================================
Files 14 13 -1
Lines 2547 2502 -45
==========================================
- Hits 2038 1992 -46
- Misses 418 420 +2
+ Partials 91 90 -1
... and 1 file with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Yes I think that’s reasonable. |
Cool, thanks. |
This PR removes let support for now as the proposal is evolving into something different jmespath/jmespath.site#6