Skip to content

add reactive-engine - #16

Merged
johnsoncodehk merged 9 commits into
johnsoncodehk:masterfrom
pravosleva:reactive-engine-exp
Oct 14, 2026
Merged

johnsoncodehk merged 9 commits into
johnsoncodehk:masterfrom
pravosleva:reactive-engine-exp

Conversation

@pravosleva

Copy link
Copy Markdown
Contributor

Hello, I would like to take part in your party.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The npm lockfile required by CI is stale, and the adapter export contains a naming typo.

2 open findings
What changed in this PR

Adds @pravosleva/reactive-engine as the sixteenth framework in the reactive behavior test matrix.

Changes:

  • Adds and registers a Reactive Engine adapter.
  • Adds dependencies and generated Yarn lock data.
  • Updates README results and framework metadata.
File Description
package.json Adds Reactive Engine and RxJS dependencies.
yarn.lock Locks the Yarn dependency graph.
test/​frameworks/​reactiveEngine.ts Implements the framework adapter.
test/​frameworks/​index.ts Registers the adapter.
README.md Updates framework counts, versions, and results.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread package.json Outdated
Comment thread test/frameworks/reactiveEngine.ts Outdated
Den Pol added 2 commits October 14, 2026 08:39
@johnsoncodehk
johnsoncodehk merged commit 9a1c472 into johnsoncodehk:master Oct 14, 2026
1 check passed
@pravosleva

pravosleva commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor Author

@johnsoncodehk could you please replace README file with @pravosleva/reactive-engine@2.2.0 (latest) ? Or need PR ?

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.

3 participants