Skip to content

[Review Only] Landing livedev2 in core#2

Closed
sebaslv wants to merge 4 commits intomasterfrom
livedev2-in-core
Closed

[Review Only] Landing livedev2 in core#2
sebaslv wants to merge 4 commits intomasterfrom
livedev2-in-core

Conversation

@sebaslv
Copy link
Owner

@sebaslv sebaslv commented Nov 3, 2014

Initial steps to land livedev2 in core. (#1 re-worked to make it more reviewable)
@busykai, please review this rather than #1

In preparation for landing livedev2 as an alternative implementation,
a folder /impls was created under LiveDevelopment folder and a the
current implementation moved to a /default folder.
Servers folder and LiveDevServerManager keep in their original location
since they will be used as shared resources for both implementations.
Make tests run after moving LiveDevelopment to /impls/default
Add a working copy of the extension as part of Brackets core under
LiveDevelopment/impls/livedev2.
To make it work need to manually change reference at brackets.js when
loading LiveDevelopment module. (/impls/default -> /impls/livedev2).
Add livedev.impl preference to let the user switch between 'default'
and 'livedev2' implementations.
@sebaslv
Copy link
Owner Author

sebaslv commented Nov 3, 2014

Closing in favor of #3 which makes copy of livedev2 files able to be reviewed.

@sebaslv sebaslv closed this Nov 3, 2014
sebaslv pushed a commit that referenced this pull request Apr 24, 2015
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.

1 participant