Skip to content
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

feat(core): adjust web clipper page #10779

Merged
merged 1 commit into from
Mar 13, 2025
Merged

feat(core): adjust web clipper page #10779

merged 1 commit into from
Mar 13, 2025

Conversation

EYHN
Copy link
Member

@EYHN EYHN commented Mar 12, 2025

No description provided.

@EYHN EYHN marked this pull request as ready for review March 12, 2025 07:05
@graphite-app graphite-app bot requested review from a team March 12, 2025 07:05
Copy link
Member Author

EYHN commented Mar 12, 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.

Copy link

codecov bot commented Mar 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.28%. Comparing base (514a5fc) to head (ecb3cf7).
Report is 2 commits behind head on canary.

Additional details and impacted files
@@            Coverage Diff             @@
##           canary   #10779      +/-   ##
==========================================
- Coverage   54.29%   54.28%   -0.01%     
==========================================
  Files        2388     2388              
  Lines      109606   109606              
  Branches    18158    18158              
==========================================
- Hits        59509    59503       -6     
+ Misses      48816    48775      -41     
- Partials     1281     1328      +47     
Flag Coverage Δ
server-test 79.33% <ø> (-0.01%) ⬇️
unittest 30.86% <ø> (ø)

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.

Copy link

graphite-app bot commented Mar 12, 2025

Merge activity

  • Mar 12, 5:27 AM EDT: A user added this pull request to the Graphite merge queue.
  • Mar 12, 9:56 AM UTC: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: '3, 2, 1 Launch', 'test-build-mobile-app / ios').
  • Mar 12, 9:37 AM EDT: A user added this pull request to the Graphite merge queue.
  • Mar 12, 2:03 PM UTC: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: '3, 2, 1 Launch', 'Legacy Blocksuite E2E Test (2)').

graphite-app bot pushed a commit that referenced this pull request Mar 12, 2025
@graphite-app graphite-app bot force-pushed the eyhn/clipper-new-signin branch from 65c353f to c2ca601 Compare March 12, 2025 09:40
@graphite-app graphite-app bot force-pushed the eyhn/clipper-new-signin branch from c2ca601 to ecb3cf7 Compare March 12, 2025 13:38
@EYHN EYHN merged commit 86729fb into canary Mar 13, 2025
67 checks passed
@EYHN EYHN deleted the eyhn/clipper-new-signin branch March 13, 2025 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants