jetstreamapp / soql-parser-js

Javascript SOQL parser
https://jetstreamapp.github.io/soql-parser-js/
MIT License
77 stars 20 forks source link

build(deps): bump chevrotain from 9.0.1 to 10.5.0 #232

Closed maxence-lefebvre closed 1 year ago

maxence-lefebvre commented 1 year ago

Hello @paustint,

I bumped chevrotain to the latest version too.

10.0.0 Dropped support for legacy ES5.1 runtimes (e.g: IE11) The minimum ECMAScript version needed to run Chevrotain is now ES2015 (ES6). This should not affect anyone running on a modern engine, meaning modern NodeJS versions or popular evergreen browsers.

Various TypeScript signatures are now more accurate and strict which could potentially cause compilation errors with some grammars implemented in TypeScript, e.g:

OPTION methods now return OUT | undefined instead of just OUT. A RULE implementation function in CstParser is now defined as returning void. The ARGS for SubruleMethodOpts options type is now better enforced via generics in the subrule definition.

https://chevrotain.io/docs/changes/BREAKING_CHANGES.html

I only had to fix some optional types, build and tests are passing green.

image

socket-security[bot] commented 1 year ago

New dependency changes detected. Learn more about Socket for GitHub ↗︎


👍 No new dependency issues detected in pull request

Bot Commands

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@* or ignore all packages with @SocketSecurity ignore-all

Pull request alert summary
Issue Status
Install scripts ✅ 0 issues
Native code ✅ 0 issues
Bin script shell injection ✅ 0 issues
Unresolved require ✅ 0 issues
Invalid package.json ✅ 0 issues
HTTP dependency ✅ 0 issues
Git dependency ✅ 0 issues
Potential typo squat ✅ 0 issues
Known Malware ✅ 0 issues
Telemetry ✅ 0 issues
Protestware/Troll package ✅ 0 issues

📊 Modified Dependency Overview:

⬆️ Updated Package Version Diff Added Capability Access +/- Transitive Count Publisher
chevrotain@10.5.0 9.0.1...10.5.0 None +4/-2 bd82
paustint commented 1 year ago

@maxence-lefebvre - Thanks for the PR! everything looks good. I will review one more time and run tests locally and if all goes well will publish a new version this afternoon.

maxence-lefebvre commented 1 year ago

Thank you ! Take your time, I am in CEST timezone so the day is almost over here