Skip to content

Conversation

@darkskygit
Copy link
Member

@darkskygit darkskygit commented Jan 9, 2025

fix PD-2167
fix PD-2169
fix PD-2190

Copy link
Member Author

darkskygit commented Jan 9, 2025


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

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

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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

@codecov
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 70.55985% with 305 lines in your changes missing coverage. Please review.

Project coverage is 54.52%. Comparing base (05f3069) to head (d8373f6).
Report is 1 commits behind head on canary.

Files with missing lines Patch % Lines
...end/server/src/plugins/copilot/context/resolver.ts 58.33% 140 Missing ⚠️
.../backend/server/src/plugins/copilot/context/job.ts 72.68% 56 Missing ⚠️
...kend/server/src/plugins/copilot/context/session.ts 74.65% 37 Missing ⚠️
...ackend/server/src/plugins/copilot/context/types.ts 66.66% 31 Missing ⚠️
...kend/server/src/plugins/copilot/context/service.ts 80.45% 17 Missing ⚠️
...nd/server/src/plugins/copilot/context/embedding.ts 60.00% 14 Missing ⚠️
...ackend/server/src/plugins/copilot/context/utils.ts 86.95% 6 Missing ⚠️
...ckages/backend/server/src/base/error/errors.gen.ts 71.42% 2 Missing ⚠️
.../core/src/blocksuite/ai/provider/copilot-client.ts 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           canary    #9611      +/-   ##
==========================================
+ Coverage   54.12%   54.52%   +0.40%     
==========================================
  Files        2413     2416       +3     
  Lines      110185   111154     +969     
  Branches    18265    18383     +118     
==========================================
+ Hits        59633    60606     +973     
- Misses      49277    49288      +11     
+ Partials     1275     1260      -15     
Flag Coverage Δ
server-test 79.75% <70.49%> (+0.46%) ⬆️
unittest 30.70% <77.77%> (+<0.01%) ⬆️

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.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch from da594ed to deb2287 Compare January 10, 2025 10:44
@github-actions github-actions bot added the test Related to test cases label Jan 10, 2025
@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch 2 times, most recently from 404e031 to b167ba4 Compare January 15, 2025 04:01
@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch 2 times, most recently from 3c5ac81 to d92f6c9 Compare January 17, 2025 07:36
@darkskygit darkskygit marked this pull request as ready for review January 17, 2025 08:03
@graphite-app graphite-app bot requested review from a team January 17, 2025 08:03
@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch 2 times, most recently from 3eb5b32 to 45ab9de Compare January 22, 2025 06:33
Copy link
Member

@akumatus akumatus left a comment

Choose a reason for hiding this comment

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

Docs and files should be stored in the same array to keep them sorted.

@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch from 45ab9de to 460e3e6 Compare January 27, 2025 04:36
@darkskygit darkskygit requested a review from akumatus January 27, 2025 05:13
@akumatus akumatus force-pushed the darksky/adapt-database-for-copilot-context branch from 6fd00a4 to 1a48583 Compare March 12, 2025 02:50
@akumatus akumatus force-pushed the darksky/doc-context-for-ai-backend branch from 2e9df69 to 8a3bcba Compare March 12, 2025 02:50
@akumatus akumatus force-pushed the darksky/adapt-database-for-copilot-context branch from 1a48583 to 5da90e6 Compare March 12, 2025 02:52
@akumatus akumatus force-pushed the darksky/doc-context-for-ai-backend branch from 8a3bcba to 4329279 Compare March 12, 2025 02:53
@darkskygit darkskygit force-pushed the darksky/adapt-database-for-copilot-context branch from 5da90e6 to 6fd00a4 Compare March 12, 2025 03:35
@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch 2 times, most recently from 9de8d20 to d478e83 Compare March 12, 2025 03:53
@darkskygit darkskygit changed the base branch from darksky/adapt-database-for-copilot-context to graphite-base/9611 March 12, 2025 11:59
@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch from ae3bed9 to 68bdb39 Compare March 12, 2025 14:44
@darkskygit darkskygit changed the base branch from graphite-base/9611 to canary March 12, 2025 14:44
@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch 6 times, most recently from 75a82ec to fbb1c95 Compare March 13, 2025 04:24
@graphite-app
Copy link
Contributor

graphite-app bot commented Mar 13, 2025

Merge activity

  • Mar 13, 4:39 AM EDT: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..
  • Mar 13, 7:42 AM EDT: A user added this pull request to the Graphite merge queue.
  • Mar 13, 8:05 AM EDT: A user merged this pull request with the Graphite merge queue.

@darkskygit darkskygit force-pushed the darksky/doc-context-for-ai-backend branch 3 times, most recently from c47adf9 to d793428 Compare March 13, 2025 11:22
fix PD-2167
fix PD-2169
fix PD-2190
@graphite-app graphite-app bot force-pushed the darksky/doc-context-for-ai-backend branch from d793428 to d8373f6 Compare March 13, 2025 11:45
@graphite-app graphite-app bot merged commit d8373f6 into canary Mar 13, 2025
67 checks passed
@graphite-app graphite-app bot deleted the darksky/doc-context-for-ai-backend branch March 13, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app:core app:server docs Improvements or additions to documentation mod:native mod:server-native rust test Related to test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants