Skip to content
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

Engine id as executionId #18

Merged
merged 1 commit into from
Dec 15, 2023
Merged

Engine id as executionId #18

merged 1 commit into from
Dec 15, 2023

Conversation

tabkram
Copy link
Owner

@tabkram tabkram commented Dec 15, 2023

No description provided.

Copy link

github-actions bot commented Dec 15, 2023

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
92.34% (+0.12% 🔼)
241/261
🟢 Branches
86.21% (+0.14% 🔼)
175/203
🟢 Functions
91.78% (+0.11% 🔼)
67/73
🟢 Lines
93.06% (+0.12% 🔼)
228/245

Test suite run success

33 tests passing in 5 suites.

Report generated by 🧪jest coverage report action from 73f539a

…nEngine

- feat: use engine.options.id when provided as ExecutionId for ExecutionEngine

- feat: make engine.options.id argument optional
@tabkram tabkram merged commit 726db14 into main Dec 15, 2023
3 checks passed
@tabkram tabkram deleted the engine-id-as-execution-id branch December 15, 2023 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant