style: enable rxjs/finnish

[EYHN]

chore(infra): use finnish notation for observables

do rename

[graphite-app]

Your org has enabled the Graphite merge queue for merging into canary

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

You can enable merging using labels in your Graphite merge queue settings.

[EYHN]

• feat(infra): livedata effect #6281
• style: enable import-x/no-duplicates #6279
• style: no import infra submodule #6278
• feat(electron): define runtimeConfig in esbuild #6287
• chore(electron): config vitest swc #6282
• style: enable rxjs/finnish #6276 👈
• canary

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @EYHN and the rest of your teammates on Graphite

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 2b42a75. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 6 targets

• nx test:coverage @affine/monorepo
• nx build @affine/storage -- --target x86_64-unknown-linux-gnu --use-napi-cross
• nx build @affine/native -- --target x86_64-pc-windows-msvc --use-napi-cross
• nx build @affine/native -- --target x86_64-apple-darwin --use-napi-cross
• nx build @affine/native -- --target aarch64-apple-darwin --use-napi-cross
• nx build @affine/native -- --target x86_64-unknown-linux-gnu --use-napi-cross

---

Sent with 💌 from NxCloud.

[codecov]

Codecov Report

Attention: Patch coverage is 39.71292% with 126 lines in your changes are missing coverage. Please review.

Project coverage is 60.05%. Comparing base (c6676fd) to head (2b42a75).

Files	Patch %	Lines
...d/core/src/modules/workbench/entities/workbench.ts	0.00%	27 Missing ⚠️
...ages/frontend/core/src/modules/tag/entities/tag.ts	0.00%	11 Missing ⚠️
...kages/frontend/core/src/modules/tag/service/tag.ts	0.00%	11 Missing ⚠️
...rc/modules/right-sidebar/entities/right-sidebar.ts	0.00%	9 Missing ⚠️
packages/common/infra/src/livedata/index.ts	83.33%	5 Missing and 1 partial ⚠️
.../core/src/modules/navigation/entities/navigator.ts	0.00%	6 Missing ⚠️
packages/common/infra/src/initialization/index.ts	0.00%	4 Missing ⚠️
...ges/common/infra/src/workspace/engine/doc/index.ts	60.00%	4 Missing ⚠️
...ontend/core/src/modules/workbench/entities/view.ts	0.00%	4 Missing ⚠️
...re/src/modules/workbench/view/split-view/panel.tsx	0.00%	4 Missing ⚠️
... and 24 more

Additional details and impacted files

@@            Coverage Diff             @@
##           canary    #6276      +/-   ##
==========================================
- Coverage   60.14%   60.05%   -0.09%     
==========================================
  Files         507      507              
  Lines       23493    23496       +3     
  Branches     2151     2150       -1     
==========================================
- Hits        14129    14110      -19     
- Misses       9103     9125      +22     
  Partials      261      261              

Flag	Coverage Δ
server-test	70.45% <0.00%> (-0.12%)	⬇️
unittest	38.59% <40.09%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[graphite-app]

Merge activity

• Mar 24, 1:04 PM EDT: EYHN added this pull request to the Graphite merge queue.
• Mar 24, 1:15 PM EDT: EYHN merged this pull request with the Graphite merge queue.