sqrl-lang / sqrl

A safe, stateful rules language for event streams
Apache License 2.0
113 stars 11 forks source link

Clean up FeatureMap in API #30

Closed qix closed 1 year ago

qix commented 1 year ago

Problem

FeatureMap is part of the API and should be cleaner.

Solution

Cleaned it up a little more, added an optional value type

Result

Just neater, noticed this snuck though. Need to be extra careful about changing the api/ folder

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 3687868882


Files with Coverage Reduction New Missed Lines %
packages/sqrl/src/api/execute.ts 10 42.53%
<!-- Total: 10 -->
Totals Coverage Status
Change from base Build 3681868395: 0.0%
Covered Lines: 15779
Relevant Lines: 24079

💛 - Coveralls
meyer commented 1 year ago

Just neater, noticed this snuck though. Need to be extra careful about changing the api/ folder

😅 noted