AFFiNE icon indicating copy to clipboard operation
AFFiNE copied to clipboard

feat(infra): new workspace infra

Open EYHN opened this issue 1 year ago • 3 comments

This PR copying @affine/workspace into common/infra, and adding definitions for services and unit tests.

EYHN avatar Jan 17 '24 17:01 EYHN

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.

graphite-app[bot] avatar Jan 17 '24 17:01 graphite-app[bot]

Current dependencies on/for this PR:

  • canary
    • PR #5562 Graphite
      • PR #5595 Graphite
      • PR #5497 Graphite
        • PR #5563 Graphite
          • PR #5564 Graphite
            • PR #5693 Graphite
              • PR #5617 Graphite 👈
                • PR #5618 Graphite
                  • PR #5565 Graphite

This stack of pull requests is managed by Graphite.

EYHN avatar Jan 17 '24 17:01 EYHN

Codecov Report

Attention: 353 lines in your changes are missing coverage. Please review.

Comparison is base (4f7e0d0) 64.63% compared to head (2e71c98) 64.26%.

Files Patch % Lines
packages/common/infra/src/workspace/engine/blob.ts 21.97% 68 Missing and 3 partials :warning:
packages/common/infra/src/workspace/upgrade.ts 26.31% 42 Missing :warning:
...s/common/infra/src/workspace/engine/sync/engine.ts 60.60% 31 Missing and 8 partials :warning:
...ges/common/infra/src/workspace/engine/sync/peer.ts 75.16% 31 Missing and 6 partials :warning:
packages/common/infra/src/workspace/list/index.ts 60.00% 24 Missing and 6 partials :warning:
packages/common/infra/src/workspace/testing.ts 62.02% 28 Missing and 2 partials :warning:
packages/common/infra/src/workspace/manager.ts 44.68% 24 Missing and 2 partials :warning:
packages/common/infra/src/utils/object-pool.ts 32.43% 24 Missing and 1 partial :warning:
packages/common/infra/src/utils/async-queue.ts 70.73% 10 Missing and 2 partials :warning:
packages/common/infra/src/workspace/workspace.ts 65.62% 10 Missing and 1 partial :warning:
... and 7 more
Additional details and impacted files
@@                 Coverage Diff                 @@
##           eyhn/fix-vitest    #5617      +/-   ##
===================================================
- Coverage            64.63%   64.26%   -0.38%     
===================================================
  Files                  308      337      +29     
  Lines                18646    19542     +896     
  Branches              1542     1676     +134     
===================================================
+ Hits                 12052    12558     +506     
- Misses                6408     6756     +348     
- Partials               186      228      +42     
Flag Coverage Δ
server-test 70.78% <ø> (ø)
unittest 46.49% <56.31%> (+2.05%) :arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jan 17 '24 17:01 codecov[bot]

Merge activity

graphite-app[bot] avatar Jan 30 '24 06:01 graphite-app[bot]