+
Skip to content

feat(store): add lazyProvider utility #2326

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

Merged
merged 1 commit into from
May 10, 2025
Merged

feat(store): add lazyProvider utility #2326

merged 1 commit into from
May 10, 2025

Conversation

arturovt
Copy link
Member

@arturovt arturovt commented May 9, 2025

No description provided.

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

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
Copy link

codeclimate bot commented May 10, 2025

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.

2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载