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

Update Electron dependencies #231

Merged
merged 9 commits into from
Jul 27, 2024
Merged

Update Electron dependencies #231

merged 9 commits into from
Jul 27, 2024

Conversation

eliandoran
Copy link
Contributor

@eliandoran eliandoran commented Jul 17, 2024

  • Neither electron-debug, nor electron-dl can be updated for now, since they have become ESM-only, causing a crash at start-up. They can be updated once we have ESM in the server (see Port server to ES modules #15).

@eliandoran eliandoran marked this pull request as draft July 17, 2024 17:46
This was linked to issues Jul 17, 2024
@eliandoran eliandoran force-pushed the feature/update_electron branch from 95d2129 to 9bc89ff Compare July 17, 2024 19:42
@eliandoran eliandoran marked this pull request as ready for review July 17, 2024 20:59
@eliandoran eliandoran requested review from a team and alexpietsch July 17, 2024 21:00
@zerebos
Copy link

zerebos commented Jul 17, 2024

I'll give it a closer look later but from a preliminary look nothing jumps out at me, this is a good change

@eliandoran eliandoran added this to the v0.90.1-beta milestone Jul 17, 2024
@eliandoran
Copy link
Contributor Author

eliandoran commented Jul 19, 2024

@rauenzi , I've "discovered" that the update of bettersqlite3 breaks backwards compatibility with Trilium Notes (see admonition in the PR description). We might want to consider the implications of that.

@zerebos
Copy link

zerebos commented Jul 19, 2024

Ah that is indeed quite a major implication... I would be hesitant to go forward with it if we're to get any initial adoption/migration of users.

Avoids an import error because electron-debug became ESM-only in version
4. We will update to it once we upgrade to ESM.
@eliandoran eliandoran force-pushed the feature/update_electron branch from 0633039 to 9db4e1d Compare July 20, 2024 12:11
@eliandoran eliandoran removed a link to an issue Jul 20, 2024
@eliandoran
Copy link
Contributor Author

eliandoran commented Jul 20, 2024

@rauenzi , OK, split the better-sqlite3 change into #250.

@TriliumNext/development , please review.

Artifacts available here: https://github.com/TriliumNext/Notes/actions/runs/10020074760

@eliandoran eliandoran changed the title Update Electron dependencies, better-sqlite3 Update Electron dependencies Jul 20, 2024
@eliandoran eliandoran marked this pull request as draft July 20, 2024 13:48
@eliandoran eliandoran modified the milestones: v0.90.1-beta, v0.90.2-beta Jul 20, 2024
Base automatically changed from feature/update_to_node20 to develop July 20, 2024 16:06
@eliandoran eliandoran modified the milestones: v0.90.2-beta, v0.90.1-beta Jul 24, 2024
@eliandoran eliandoran merged commit fd1f43d into develop Jul 27, 2024
6 of 7 checks passed
@eliandoran eliandoran deleted the feature/update_electron branch July 27, 2024 12:20
@eliandoran eliandoran restored the feature/update_electron branch December 14, 2024 06:56
@eliandoran eliandoran deleted the feature/update_electron branch December 14, 2024 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Electron dependencies
2 participants