Closed
Bug 1299203
Opened 8 years ago
Closed 8 years ago
Stop using a submodule for UI
Categories
(Release Engineering Graveyard :: Applications: Balrog (backend), defect)
Release Engineering Graveyard
Applications: Balrog (backend)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jlorenzo, Assigned: jlorenzo)
References
Details
Attachments
(2 files)
Discussed with :bhearsum over IRC. With the current docker image implementation, we can easily manually test Balrog from an end to another. Also, having the UI in its own repo makes changes that affect both front-end and back-end harder.
We can now put the UI in the main repo, without using a submodule. One way to achieve that could be to transfer the commits from one repo to another. See [1] for more details about the implementation.
Side note: Let's wait until the number of active PRs reaches 0. Otherwise, they'll have to be heavily edited before creating a new PR in the main repo.
[1] https://github.com/fxbox/foxbox/pull/406#issuecomment-212449454
Assignee | ||
Comment 1•8 years ago
|
||
One request from :bhearsum, on IRC:
> i'd prefer that everything prior to
> https://github.com/mozilla/balrog-ui/commit/5e70edaa020cf86d8d2962052857b3db773eeabe
> is squashed together - most of the stuff before that is early development, and
> i don't think there's any value to preserving its commit-by-commit history
Assignee | ||
Comment 2•8 years ago
|
||
I found the reason of the tests hanging. Lineman delegates the test execution to testem. Per [1], it must be present in node_modules/.bin. That wasn't the case in the Docker image. I added it to packages.json.
I also added "phantom-prebuilt". This packages makes sure PhantomJS is in $PATH by downloading it.
Tests are now running and passing [2].
Review suggestions:
* Look at the commit tree
* See diff for HEAD and HEAD~2 (HEAD~1 being the merge commit).
[1] https://github.com/linemanjs/lineman/blob/3f828e5b96b72c6a9d562a0dba23e8f2cd4d2cfc/lib/testem-utils.coffee#L5
[2] https://tools.taskcluster.net/task-inspector/#IC47ho9VRvKti6Crp1p59w/0
Attachment #8790216 -
Flags: review?(bhearsum)
Assignee | ||
Comment 3•8 years ago
|
||
Review suggestions:
* every file but .gitignore .gitkeep and README.md are deleted
* .gitignore filters out everything
* .gitkeep still let README.md be in the repo
* README.md points people to the new location of the UI (which doesn't exist until the balrog PR is merged)
Attachment #8790223 -
Flags: review?(bhearsum)
Updated•8 years ago
|
Attachment #8790223 -
Flags: review?(bhearsum) → review+
Updated•8 years ago
|
Attachment #8790216 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 4•8 years ago
|
||
Barog: https://github.com/mozilla/balrog/commit/6f35e8c2aa0069bce7f3a5b1a173a78e910148b9
Balrog UI: https://github.com/mozilla/balrog-ui/commit/9ff2560647a5be0122a7b013542f80129302c795
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Product: Release Engineering → Release Engineering Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•