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

Implementing download-edge to automate downloading and unzipping Edge files #167

Merged
merged 7 commits into from
Jul 1, 2020

Conversation

mliao95
Copy link

@mliao95 mliao95 commented Jun 9, 2020

Fixed file issue in https://thirdpartysource.microsoft.com/, removing unnecessary code associated with accommodating the directory structure issue with the previously uploaded ZIPs.

Adding section for Windows users to use the bash script to download Edge files.

@mliao95 mliao95 force-pushed the user/milia/fix-unzip-script branch 2 times, most recently from ff679cd to bcda5e7 Compare June 9, 2020 23:25
@mliao95 mliao95 linked an issue Jun 9, 2020 that may be closed by this pull request
@mliao95 mliao95 requested a review from vidorteg June 9, 2020 23:27
* `[ZIP_FILE]:\src\out\Release\gen\devtools`
* copy the contents of the "devtools" folder and paste them into `c:\edge\src\out\Release\gen\devtools`
* Set the `EDGE_CHROMIUM_PATH` environment variable to `c:\edge\src` (assuming your drive is located at C:\)
* Download and Extract Edge source files
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vidorteg , when you get the chance, could you try out these new contributing steps and give me any feedback? Thanks

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the idea of having this process automated, but I would prefer to use npm instead of a new bash, users already know javascript so let's make contribution even easier.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. PTAL

Copy link
Contributor

@vidorteg vidorteg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some comments

@mliao95 mliao95 force-pushed the user/milia/fix-unzip-script branch from 9190712 to bfbaf9c Compare June 17, 2020 00:47
@mliao95 mliao95 changed the title Updating unzipping in pipeline and adding unzipping script to contributing doc Implementing download-edge to automate downloading and unzipping Edge files Jun 17, 2020
@mliao95 mliao95 force-pushed the user/milia/fix-unzip-script branch from 2ad5c37 to e9f3872 Compare June 17, 2020 00:56
Copy link
Contributor

@vidorteg vidorteg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

Remove code for old directory support in unzipping scripts
2 participants