appengine/swarming/elements/README.md - Issue 2174903002: Add stub pages and oauth-blessed request to /newui

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: appengine/swarming/elements/README.md

Issue 2174903002: Add stub pages and oauth-blessed request to /newui (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-py@master

Patch Set: more docs Created 4 years, 5 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

OLD	NEW
1 This contains all the Polymer 1.X elements used in swarming.	1 This contains all the Polymer 1.X elements used in swarming.

	2

	3 To clean and build the pages for deploying, run

	4

	5 make vulcanize

	6

	7 This combines all of the elements needed to display the page into several "singl e-page" apps, like

	8 the bot-list.

	9 These are checked into version control so that they may be easily redeployed w/o having to rebuild
	stephana 2016/07/24 21:17:06 IMO, checking the combined apps into source contro IMO, checking the combined apps into source control is error prone. Repeated deployment of the same app can be simplified by the deployment system, i.e. similar to the mechanism that skia-buildbot uses with push and Debian packages. kjlubick 2016/07/25 12:45:43 I would also prefer what you suggested. M-A has r Show quoted text On 2016/07/24 at 21:17:06, stephana wrote: > IMO, checking the combined apps into source control is error prone. Repeated deployment of the same app can be simplified by the deployment system, i.e. similar to the mechanism that skia-buildbot uses with push and Debian packages. I would also prefer what you suggested. M-A has requested it to be this way. stephana 2016/07/25 13:26:32 Acknowledged. Show quoted text On 2016/07/25 12:45:43, kjlubick wrote: > On 2016/07/24 at 21:17:06, stephana wrote: > > IMO, checking the combined apps into source control is error prone. Repeated > deployment of the same app can be simplified by the deployment system, i.e. > similar to the mechanism that skia-buildbot uses with push and Debian packages. > > I would also prefer what you suggested. M-A has requested it to be this way. Acknowledged.
	10 the pages if there were no changes.

	11

	12

	13 To incrementally re-build for local testing, run

	14

	15 make quick_vulcanize

	16

	17

2 To access the demo pages on localhost:8080, run	18 To access the demo pages on localhost:8080, run

3	19

4

5 make && make run	20 make && make run

6	21

7	22

8 You will need to install node.js, npm, and bower, for example:	23 You will need to install node.js, npm, and bower, for example:

9	24

10 sudo apt-get install npm nodejs-legacy	25 sudo apt-get install npm nodejs-legacy

11 sudo npm install -g bower	26 sudo npm install -g bower

	27

OLD	NEW

« appengine/swarming/elements/Makefile ('K') | « appengine/swarming/elements/Makefile ('k') | appengine/swarming/elements/botlist.html » ('j') | appengine/swarming/elements/bower.json » ('J')