Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29890593: Noissue - Enable test runner to run on Windows and add documentation (Closed)

Created:
Sept. 24, 2018, 2:45 p.m. by Sebastian Noack
Modified:
Sept. 25, 2018, 5:11 p.m.
Reviewers:
tlucas
CC:
geo, rossg
Visibility:
Public.

Description

Noissue - Enable test runner to run on Windows and add documentation

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fixed typo #

Patch Set 3 : Use double quotes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -8 lines) Patch
M README.md View 1 3 chunks +22 lines, -4 lines 0 comments Download
M package.json View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
M test/all.js View 1 2 2 chunks +4 lines, -2 lines 0 comments Download

Messages

Total messages: 4
Sebastian Noack
We cannot run "npm test" from within the Linux environment on Windows, since Selenium would ...
Sept. 24, 2018, 2:49 p.m. (2018-09-24 14:49:51 UTC) #1
tlucas
Can you please file an additional merge request on gitlab.com for this? Not as a ...
Sept. 25, 2018, 12:25 p.m. (2018-09-25 12:25:15 UTC) #2
Sebastian Noack
There you go: https://gitlab.com/eyeo/adblockplus/adblockpluschrome/pipelines/31168869 Note that this branch also includes the other two changes to ...
Sept. 25, 2018, 4:21 p.m. (2018-09-25 16:21:25 UTC) #3
tlucas
Sept. 25, 2018, 5:03 p.m. (2018-09-25 17:03:44 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld