-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Override existing asar symlink in postinstall #3369
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
Conversation
This ensures the link is correct. Should fix coder#3355.
Codecov Report
@@ Coverage Diff @@
## main #3369 +/- ##
=======================================
Coverage 58.95% 58.95%
=======================================
Files 35 35
Lines 1703 1703
Branches 374 374
=======================================
Hits 1004 1004
Misses 561 561
Partials 138 138 Continue to review full report at Codecov.
|
@@ -56,18 +56,21 @@ main() { | |||
fi | |||
} | |||
|
|||
# This is a copy of symlink_asar in ../lib.sh. Look there for details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we import that lib and share code instead of duplicating it? we have a general problem (especially in our other projects) that we have unnecessary copies of things, and that often results in inconsistencies between those files over time
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that makes a lot of sense, in fact they were already out of sync with this version using -e
and the other using -L
. I opened #3371
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quick fix! Nice work 👏
So what's up with all of this |
Sorry what's up Codcov |
This ensures the link is correct. Should fix #3355.