Skip to content

Conversation

@arturovt
Copy link
Member

@arturovt arturovt commented May 9, 2025

No description provided.

@nx-cloud
Copy link

nx-cloud bot commented May 9, 2025

View your CI Pipeline Execution ↗ for commit 0808aca.

Command Status Duration Result
nx run-many --target=test --all --configuration... ✅ Succeeded 2m 15s View ↗
nx run-many --target=lint --all --exclude=creat... ✅ Succeeded 6s View ↗
nx lint-types store ✅ Succeeded <1s View ↗
nx run-many --target=build --all ✅ Succeeded 1m 29s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-10 14:55:50 UTC

@pkg-pr-new
Copy link

pkg-pr-new bot commented May 9, 2025

Open in StackBlitz

@ngxs/devtools-plugin

npm i https://pkg.pr.new/@ngxs/devtools-plugin@2326

@ngxs/form-plugin

npm i https://pkg.pr.new/@ngxs/form-plugin@2326

@ngxs/hmr-plugin

npm i https://pkg.pr.new/@ngxs/hmr-plugin@2326

@ngxs/router-plugin

npm i https://pkg.pr.new/@ngxs/router-plugin@2326

@ngxs/storage-plugin

npm i https://pkg.pr.new/@ngxs/storage-plugin@2326

@ngxs/store

npm i https://pkg.pr.new/@ngxs/store@2326

@ngxs/websocket-plugin

npm i https://pkg.pr.new/@ngxs/websocket-plugin@2326

commit: 0808aca

@markwhitfeld
Copy link
Member

Nice! Let's also add some tests and documentation for this utility 👍

@arturovt arturovt force-pushed the feat/lazy-provider branch from a1b035f to 0808aca Compare May 10, 2025 14:49
@arturovt arturovt marked this pull request as ready for review May 10, 2025 14:49
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 0808aca and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 96.3% (50% is the threshold).

This pull request will bring the total coverage in the repository to 94.0% (0.0% change).

View more on Code Climate.

@arturovt arturovt merged commit feaa76f into master May 10, 2025
12 checks passed
@arturovt arturovt deleted the feat/lazy-provider branch May 10, 2025 15:54
@markwhitfeld
Copy link
Member

Fantastic work!!!!

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