Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

fix(scripts): Fix the run_remote_dev.sh script #6505

Merged
merged 1 commit into from Aug 27, 2018
Merged

fix(scripts): Fix the run_remote_dev.sh script #6505

merged 1 commit into from Aug 27, 2018

Conversation

hritvi
Copy link
Contributor

@hritvi hritvi commented Aug 27, 2018

scripts/run_remote_dev.sh tried to run grunt watch sass which no longer exists because of which the server didn't stop listening even after exiting by CTRL+C.
Problem found in #6439

@hritvi
Copy link
Contributor Author

hritvi commented Aug 27, 2018

@shane-tomlinson the tests failed because of time-out. I can't figure out why is this happening. Maybe re-running the tests might help?

@shane-tomlinson
Copy link

This looks good to me, waiting for the tests to all turn green. Not sure what that test timeout is caused by, works great locally.

Copy link

@shane-tomlinson shane-tomlinson left a comment

Choose a reason for hiding this comment

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

r+, thank you @hritvi!

@shane-tomlinson shane-tomlinson merged commit 05d10b8 into mozilla:master Aug 27, 2018
@shane-tomlinson shane-tomlinson added this to the FxA-0: quality milestone Sep 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants