-
-
Notifications
You must be signed in to change notification settings - Fork 31
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(build): Adds automated release package creation script #388
feat(build): Adds automated release package creation script #388
Conversation
90f34cd
to
807d22d
Compare
807d22d
to
e92ac27
Compare
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. I fear and loathe bash, so I wound up insulting it a bit and adding a number of comments. Easy stuff though. LGTM once you're ready.
scripts/build.sh
Outdated
mkdir -p build/release/ | ||
|
||
# 4. Clean up any existing release zip | ||
rm -f build/release/$browserType-release.zip |
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.
Do you need to force this? Scary stuff.
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.
Do you need to force this?
Not really, I used the -f
flag to suppress the exit code when the file couldn't be found. However, I've modified the command to handle missing files differently. Thanks for your comment.
This pull request introduces a new script,
build.sh
, to address inconsistencies in how Chrome and Firefox handle thefavicon_url
key within thesearch_provider
object. This script automates the creation of release packages specific to each browser.Key changes:
Ensures consistent behavior across Chrome and Firefox by addressing the
favicon_url
inconsistency.Introduces two new commands in
package.json
:npm run release-chrome
: Creates a Chrome-compatible release package within a dedicated build folder.npm run release-firefox
: Creates a Firefox-compatible release package within the same build folder.Removes the manual creation of
zip
files, streamlining the release process.Updates the
README.md
file to reflect new build steps