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

fix: try to sign macos #2066

Merged
merged 8 commits into from
Apr 21, 2023
Merged

fix: try to sign macos #2066

merged 8 commits into from
Apr 21, 2023

Conversation

pengx17
Copy link
Collaborator

@pengx17 pengx17 commented Apr 21, 2023

sqlite3 seems not working properly to be bundled in CI. This PR fix it by replace with better-sqlite3.

πŸ€– Generated by Copilot at 4eafb2a

Summary

πŸš€πŸ› οΈπŸ›

This pull request improves the app's performance, reliability, and compatibility by switching to better-sqlite3 for SQLite database operations, enhancing error handling and logging, supporting rebuilding native modules, and using the ULFO format for Linux packaging. It also updates the dependencies and configuration files accordingly.

We forge the ULFO, the ultimate Linux tool
We rebuild the native modules, we don't follow the rules
We switch to better-sqlite, we optimize the speed
We handle the errors, we log them as we bleed

Walkthrough

  • Switch from sqlite3 to better-sqlite3 module for SQLite database operations (link,link,link,link,link,link,link,link,link)
  • Enable ULFO format for Linux distribution using Electron Forge (link)
  • Add electron-rebuild script to rebuild native modules (link)
  • Exclude yjs module from bundling process (link)
  • Handle and log filesystem errors in workspace operations (link,link)
  • Move electron-log module to devDependencies (link)

@vercel
Copy link

vercel bot commented Apr 21, 2023

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
affine-preview βœ… Ready (Inspect) Visit Preview Apr 21, 2023 2:12pm
affine-storybook βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Apr 21, 2023 2:12pm

@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:29 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:29 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:29 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:29 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:29 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:29 — with GitHub Actions Inactive
@github-actions github-actions bot added the app:electron Related to electron app label Apr 21, 2023
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:30 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:31 — with GitHub Actions Inactive
@vercel vercel bot temporarily deployed to Preview – affine-storybook April 21, 2023 11:32 Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:33 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.05 πŸŽ‰

Comparison is base (f66d402) 74.09% compared to head (4eafb2a) 74.14%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2066      +/-   ##
==========================================
+ Coverage   74.09%   74.14%   +0.05%     
==========================================
  Files         299      299              
  Lines        5930     5930              
  Branches     1089     1082       -7     
==========================================
+ Hits         4394     4397       +3     
+ Misses       1314     1311       -3     
  Partials      222      222              
Flag Coverage Ξ”
e2etest 72.85% <ΓΈ> (+0.08%) ⬆️
server-test 90.78% <ΓΈ> (ΓΈ)
storybook-test 49.05% <ΓΈ> (ΓΈ)
unittest 37.61% <ΓΈ> (+0.01%) ⬆️

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

see 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

β˜” View full report in Codecov by Sentry.
πŸ“’ Do you have feedback about the report comment? Let us know in this issue.

@cloudflare-pages
Copy link

cloudflare-pages bot commented Apr 21, 2023

Deploying with Β Cloudflare Pages Β Cloudflare Pages

Latest commit: 4eafb2a
Status:Β βœ…Β  Deploy successful!
Preview URL: https://46d76a89.affine.pages.dev
Branch Preview URL: https://fix-sign-macos.affine.pages.dev

View logs

@vercel vercel bot temporarily deployed to Preview – affine-preview April 21, 2023 11:36 Inactive
@perfsee
Copy link

perfsee bot commented Apr 21, 2023

affine-toeverything

Bundle main

diff ------------------- Bundle Size Diff -------------------------

@@                       EntryPoint: main                        @@
##                     master fix/sign-macos                 +/- ##
===================================================================
< Bundle              4.26 MB        4.26 MB       +71 B(+0.00%)   
< Initial JS          1.74 MB        1.74 MB       +71 B(+0.00%)   
= Initial CSS         13.2 kB        13.2 kB                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
= Assets                   43             43                       
= Chunks                   41             41                       
= Packages                150            150                       
= Duplicates                5              5                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Warnings ~~~~~~~~~~~~~~~~~~~~~~~~~~~#
! Deduplicate versions of libraries                                
! Separate mixed content assets files                              
! Avoid non-minified assets                                        
! Avoid cache wasting                                              

@@                    EntryPoint: pages/_app                     @@
##                     master fix/sign-macos                 +/- ##
===================================================================
< Bundle              4.15 MB        4.15 MB       +71 B(+0.00%)   
< Initial JS          1.64 MB        1.64 MB       +71 B(+0.00%)   
= Initial CSS         13.2 kB        13.2 kB                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
= Assets                   41             41                       
= Chunks                   39             39                       
= Packages                150            150                       
= Duplicates                5              5                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Warnings ~~~~~~~~~~~~~~~~~~~~~~~~~~~#
! Deduplicate versions of libraries                                
! Separate mixed content assets files                              
! Avoid non-minified assets                                        
! Avoid cache wasting                                              

@@                   EntryPoint: pages/_error                    @@
##                     master fix/sign-macos                 +/- ##
===================================================================
= Bundle              16.7 kB        16.7 kB                       
= Initial JS          16.7 kB        16.7 kB                       
= Initial CSS             0 B            0 B                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
= Assets                    2              2                       
= Chunks                    2              2                       
= Packages                  5              5                       
= Duplicates                0              0                       

...and more

@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:41 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:41 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:41 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 11:41 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 13:59 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 13:59 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 13:59 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 13:59 — with GitHub Actions Inactive
@vercel vercel bot temporarily deployed to Preview – affine-preview April 21, 2023 14:00 Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:04 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:04 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:04 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:04 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:04 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:04 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:04 — with GitHub Actions Inactive
@vercel vercel bot temporarily deployed to Preview – affine-storybook April 21, 2023 14:07 Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:08 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:09 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:09 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:09 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:09 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:10 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:10 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:10 — with GitHub Actions Inactive
@pengx17 pengx17 temporarily deployed to development April 21, 2023 14:10 — with GitHub Actions Inactive
@vercel vercel bot temporarily deployed to Preview – affine-preview April 21, 2023 14:12 Inactive
@pengx17 pengx17 requested a review from himself65 April 21, 2023 14:25
@pengx17 pengx17 enabled auto-merge (squash) April 21, 2023 14:26
@pengx17 pengx17 requested a review from tzhangchi April 21, 2023 15:30
@pengx17 pengx17 merged commit 7094385 into master Apr 21, 2023
@pengx17 pengx17 deleted the fix/sign-macos branch April 21, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:electron Related to electron app
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants