-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Create mach fetch
command
#12821
Create mach fetch
command
#12821
Conversation
Heads up! This PR modifies the following files:
|
@@ -34,6 +34,17 @@ def cargo(self, params): | |||
if not params: | |||
params = [] | |||
|
|||
cargo_paths = [path.join('components', 'servo'), |
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.
These are also used in update_cargo
, so this can be moved to the top level as a constant. Also, formatting nit, put each path on a new line with a trailing comma:
CARGO_PATHS = [
path.join(...),
]
Addressed all reviews. |
LGTM. r? @aneeshusa |
CARGO_PATHS = [ | ||
path.join('components', 'servo'), | ||
path.join('ports', 'cef'), | ||
path.join('ports', 'geckolib') |
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.
nit: add a trailing comma at the end of the line
LGTM aside from the one nit. |
done. |
@bors-servo r+ |
📌 Commit d805e85 has been approved by |
Create `mach fetch` command - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12775. r? @larsbergstrom <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12821) <!-- Reviewable:end -->
☀️ Test successful - arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev |
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsmach update
command to update everything needed to build #12775.r? @larsbergstrom
This change is