-
-
Notifications
You must be signed in to change notification settings - Fork 221
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
fix(core): fixed jsonpath exprs may occasionally yield incorect results #1917
Conversation
@@ -32,13 +32,13 @@ | |||
"ajv-errors": "~3.0.0", | |||
"ajv-formats": "~2.1.0", | |||
"blueimp-md5": "2.18.0", | |||
"expression-eval": "4.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expression-eval
uses an older version of jsep
There's a PR there that adds newer jsep
, but until it's merged we can use simple-eval
which does roughly the same.
It's my library tho, but it's very small. Here's pretty much the entire code https://github.com/P0lip/simple-eval/blob/master/src/reduce.mj
'./__karma__/**/*.ts': ['karma-typescript'], | ||
}, | ||
|
||
// @ts-expect-error: non-standard - karma-env-preprocessor | ||
envPreprocessor: ['USE_NIMMA'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this ENV var is no longer used, should have been deleted long ago.
# [@stoplight/spectral-core-v1.6.1](https://github.com/stoplightio/spectral/compare/@stoplight/spectral-core-v1.6.0...@stoplight/spectral-core-v1.6.1) (2021-11-04) ### Bug Fixes * **core:** fixed jsonpath expressions may occasionally yield incorect results ([#1917](#1917)) ([1a5a227](1a5a227))
🎉 This PR is included in version @stoplight/spectral-core-v1.6.1 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
Fixes #1910
Checklist
Does this PR introduce a breaking change?
Interesting changes