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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: clean up the default app, add CSP #13437
Conversation
tools/js2asar.py
Outdated
@@ -24,6 +24,10 @@ def main(): | |||
def copy_files(source_files, output_dir): | |||
for source_file in source_files: | |||
output_path = os.path.join(output_dir, source_file) | |||
# Files that aren't in the default_app folder need to be put inside | |||
# the temp one we are making so they end up in the ASAR | |||
if source_file[:12] != "default_app/": |
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.
!source_file.startswith("default_app/")
would be a little cleaner, I think.
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.
Updated 馃憤
Do we want to commit the package lock here? |
@codebytere Yeah, an extra dependency was added so the lock was updated 馃憤 |
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.
Looks good to me 馃憤
e22cdfa
to
a55f34c
Compare
I got fed up seeing the CSP warnings when launching the default app 馃槃
This PR: