Jakob Ackermann
35d64a1fba
Merge pull request #3020 from overleaf/jpa-harden-copying-of-makefiles
...
[misc] make: harden parallel copying of the Makefile.module
GitOrigin-RevId: 2612516ebf1e33022233ad19741e5c7cda014483
2020-07-25 02:06:25 +00:00
Miguel Serrano
d7ff985554
Merge pull request #3018 from overleaf/jpa-retry-webpack-build
...
[misc] work around flaky webpack builds: retry once before failing
GitOrigin-RevId: 3247c2ae8f185e269b17f64c3b5d42cf16f96a5c
2020-07-17 02:07:24 +00:00
Miguel Serrano
b573374d0e
Merge pull request #2950 from overleaf/as-jsdom-tests
...
JSDOM tests
GitOrigin-RevId: d9e39c61fb2028c5b5577488d74720ef2985eebc
2020-07-11 02:04:35 +00:00
Jakob Ackermann
48b138f227
Merge pull request #2929 from overleaf/jpa-ci-3m30s-with-custom-cloudbuild
...
[misc] another batch of CI improvements
GitOrigin-RevId: 470fd762ba541d5dce92f0e99830fb2fd8fc4747
2020-07-03 02:04:13 +00:00
Jakob Ackermann
2a8e2efe62
Merge pull request #2845 from overleaf/jpa-faster-web-ci
...
[misc] faster web ci
GitOrigin-RevId: bbd6b12040956f6b397c6ea4d8144e733299400b
2020-05-23 03:18:08 +00:00
Jakob Ackermann
35400de80a
Merge pull request #2766 from overleaf/jpa-make-cleanup
...
[make] cleanup dc networks and fix frontend target
GitOrigin-RevId: abcd0d061bd924ff0ddf885d1a18b66ebcc7e8e5
2020-04-23 03:29:34 +00:00
Alasdair Smith
db610f7efb
Merge pull request #2657 from overleaf/revert-2650-revert-2570-as-sentry-source-maps
...
Bring back upload source maps to Sentry
GitOrigin-RevId: 22024e3ec2462d0bc9ad9c14e8d1c21880c55cd1
2020-04-01 03:21:16 +00:00
Timothée Alby
d4fc866984
Merge pull request #2658 from overleaf/jpa-import-makefile-changes
...
[misc] import Makefile changes
GitOrigin-RevId: 6b19abd48cc40bdbaab6b0005711c9c2cca292f7
2020-03-10 04:14:29 +00:00
Alasdair Smith
ad361a6617
Merge pull request #2650 from overleaf/revert-2570-as-sentry-source-maps
...
Revert "Upload source maps to Sentry"
GitOrigin-RevId: 6f6a122209553b9fde664ff55ebec37f7cb6ef7e
2020-03-04 04:21:39 +00:00
Eric Mc Sween
a11a4ac614
Merge pull request #2570 from overleaf/as-sentry-source-maps
...
Upload source maps to Sentry
GitOrigin-RevId: badbce7a5a72c3c2b5ad61d1ab2e458a52128e37
2020-03-04 04:21:34 +00:00
Alasdair Smith
26190da566
Merge pull request #2349 from overleaf/as-webpack-css
...
Compile CSS using webpack
GitOrigin-RevId: 489834038667dde572ea5d9b4b9392b362259a09
2019-11-28 10:39:09 +00:00
Eric Mc Sween
082f790fb6
Merge pull request #2180 from overleaf/as-clean-up-makefile
...
Clean up how tests are run in Makefile
GitOrigin-RevId: 9b52ead16392cca09bdf7edc4365f10dc468ccf2
2019-10-23 12:58:39 +00:00
Alasdair Smith
9cf73f965c
Merge pull request #1440 from overleaf/as-amd-webpack
...
Bundle all frontend code with webpack
GitOrigin-RevId: 1bd93dad516c456fe1649193868e841e20459b0b
2019-10-16 10:36:21 +00:00
Timothée Alby
7785c7b59e
Merge pull request #2029 from overleaf/ta-build-main-recurly-3
...
Load Recurly JS Separately
GitOrigin-RevId: 8baa2be76fa9d8a47676bc0bd0a80f4c05cc1600
2019-07-31 15:44:44 +00:00
Timothée Alby
8fb6835f8a
Merge pull request #2026 from overleaf/ta-build-main-recurly
...
Add build steps for main-recurly.js
GitOrigin-RevId: 25231c62854b75904b1b5288b3a5157e10220f9d
2019-07-31 09:49:07 +00:00
Alasdair Smith
0ca81de78c
Merge pull request #1717 from overleaf/as-decaffeinate-backend
...
Decaffeinate backend
GitOrigin-RevId: 4ca9f94fc809cab6f47cec8254cacaf1bb3806fa
2019-05-29 09:32:21 +00:00
Simon Detheridge
deb9e656a9
Merge pull request #1801 from overleaf/ew-module-unit-test
...
add ability to unit test a single module
GitOrigin-RevId: 584708066dea3c8d6afa1483953bbae252b48fa7
2019-05-23 13:25:39 +00:00
Ersun Warncke
6513cc7b7a
Revert "Revert "complete v2 collabratec api""
...
This reverts commit 378825fae46cef6f19c8f908a1d6cf9c837cd1d6.
GitOrigin-RevId: cf38aa3b4ec53ec6741389885ebe882476cb9297
2019-05-14 10:24:11 +00:00
Ersun Warncke
4189f2e6ec
Revert "complete v2 collabratec api"
...
This reverts commit 9c75b657c11f98e3f927dd01a422157f53a5672d.
GitOrigin-RevId: 378825fae46cef6f19c8f908a1d6cf9c837cd1d6
2019-05-13 15:52:25 +00:00
Ersun Warncke
65cbfbaeae
complete v2 collabratec api
...
* remove proxies to v1
* allow oauth tokens with user stubs
* add profile/unlink end points
* fix Makefile issue
GitOrigin-RevId: 9c75b657c11f98e3f927dd01a422157f53a5672d
2019-05-13 12:05:04 +00:00
Timothée Alby
8187eb7fcb
Merge pull request #1592 from sharelatex/ew-clean-stray-containers
...
Clean Up Containers
GitOrigin-RevId: 334f5458981b68432f0e0d086176f508ee83e836
2019-03-11 11:07:59 +00:00
Henry Oswald
68c7cc6fa3
Merge pull request #1577 from sharelatex/ho-jenkins-github-connection
...
point to the web-sharelatex-internal repo for github-> jenkins
GitOrigin-RevId: 9f31bed9296181484baf9dc11d488763f1b57b50
2019-03-11 11:04:10 +00:00
Jessica Lawshe
0d4c92054b
Merge pull request #1557 from sharelatex/ew-update-clean
...
add network/image prune for clean_ci
GitOrigin-RevId: 23be6f849c0c0fd473a76c12e5ba03565f1f3808
2019-03-11 11:02:32 +00:00
Ersun Warncke
2421593d85
run unit and acceptance tests in parallel ( #1545 )
...
GitOrigin-RevId: fefdfa04d9b9496bf6c98b37331fc4a084de065a
2019-03-11 11:00:58 +00:00
Henry Oswald
d55b38cc05
Merge pull request #1471 from sharelatex/ho-docker2
...
Move to docker image based builds
GitOrigin-RevId: 992aeaf69bfea4d1847a07c457575a63985d4407
2019-03-11 10:59:20 +00:00
Alasdair Smith
9b97af8977
Merge pull request #1403 from sharelatex/as-swap-brand-prefix
...
Swap brand prefix so OL is the default
GitOrigin-RevId: 60f4f03598fb6befc1ce790d39f546490612a1db
2019-02-12 11:32:28 +00:00
James Allen
b85595b8ca
Merge pull request #1426 from sharelatex/as-cleanup-grunt
...
Cleanup grunt config & dependencies
GitOrigin-RevId: e44445e727e8cf56d03bf33a8741b9a47418a620
2019-01-29 10:12:10 +00:00
Brian Gough
0879226f74
Merge pull request #1330 from sharelatex/as-fix-prettier-ignore
...
Fix linting ignoring index.js files in modules
GitOrigin-RevId: 8ac9fa8f8655553680678b206848d1a66718d821
2019-01-15 10:55:36 +00:00
James Allen
65c1943c63
Merge pull request #1249 from sharelatex/ja-single-process-acceptance-tests-2
...
Move to running acceptance tests in same process as web server
GitOrigin-RevId: 65757f0b57ac621cafb0066b5c5a7abb30333edd
2018-12-13 09:25:50 +00:00
Douglas Lovell
aae0484458
Merge branch 'master' into dcl-i1207
...
GitOrigin-RevId: c947041ca99860d4afb62ecfd28ba6fe1c717bfc
2018-11-30 13:24:27 +00:00
Douglas Lovell
ed3147a58f
Allow open from template button when lacking source
...
GitOrigin-RevId: 8fd49bff6cc0f66d041bb70f9345b2744978be3a
2018-11-30 13:24:23 +00:00
Jessica Lawshe
e8b7ab1f8a
Merge pull request #1195 from sharelatex/jel-portal-ieee
...
Add IEEE portal
GitOrigin-RevId: 7f3ba43be0874791cfd32bb8bf3004aac61b22d3
2018-11-29 15:27:09 +00:00
Paulo Jorge Reis
02eb683298
Merge pull request #1113 from sharelatex/pr-compile-only-ol-dark-dev
...
Move LESS compilation process to Makefile
GitOrigin-RevId: 5a3871759eda2685ec0690899b0904d45c7db0a6
2018-11-16 12:02:47 +00:00
Alasdair Smith
659242b457
Merge pull request #1037 from sharelatex/as-decaffeinate
...
Decaffeinate frontend
GitOrigin-RevId: 1c8c53dedecfe55f9936a13408df17b852f996de
2018-11-05 10:19:25 +00:00
Paulo Reis
2ef1b4904d
Bootstrap the light theme.
2018-07-09 15:45:09 +01:00
James Allen
99728d8f18
Bail out with exit code on acceptance test failure
2018-06-19 13:56:34 +01:00
James Allen
c60b74a2b1
Merge pull request #652 from sharelatex/ja-fix-front-end-test
...
Fix frontend HistoryV2ManagerTests
2018-06-14 09:11:32 +01:00
James Allen
76d10e6402
Improve error message readability
2018-06-13 10:40:36 +01:00
James Allen
f57290382d
Explain error if web crashes in acceptance tests
2018-06-13 10:11:38 +01:00
James Allen
d4efdb3ede
Fix frontend HistoryV2ManagerTests
2018-06-11 15:47:23 +01:00
Hayden Faulds
be0fd9a446
reduce container teardown timeout to 0
2018-06-05 11:30:48 +01:00
Tim Alby
a9f1989eca
add test_unit_app task
2018-05-23 15:47:28 +02:00
James Allen
b1f378208d
Properly merge subscriptions from different places
2018-05-22 10:05:08 +01:00
James Allen
1e8439a2c6
Fix restore of docs from old deleted docs
2018-04-06 10:30:13 +01:00
Alasdair Smith
4d79c68f41
Rename webpack prod compile to minify_es and run as part of minify step
...
Fixes issue where webpack prod files were wiped after running minify on CI
2018-03-15 11:29:36 +00:00
Alasdair Smith
8537c7b9b6
Add production webpack npm script
2018-03-15 09:42:26 +00:00
Alasdair Smith
864bf771a4
Merge pull request #362 from sharelatex/as-linting
...
Linting new ES code
2018-02-27 11:06:11 +00:00
Brian Gough
bc37003769
use COFFEE_OPTIONS by default
2018-02-26 09:50:23 +00:00
Brian Gough
42b701fd34
use the make automatic variable @D for the target directory name
2018-02-23 11:43:10 +00:00
Alasdair Smith
325f91ed2e
Add scripts for running linting
2018-02-23 11:41:21 +00:00
Brian Gough
d364bfc353
add support for debugging
2018-02-23 11:36:08 +00:00
Alasdair Smith
f529137db0
Clean frontend tests
2018-02-20 14:01:50 +00:00
Alasdair Smith
24ec7b77ab
Convert frontend test runs to use Docker/karma
2018-02-20 14:01:50 +00:00
Alasdair Smith
09fbafa1f6
Include ShareJS CM adapter in compiled lib and fix bug with CM adapter
...
Because the CM & Ace scripts are naively concat-ed together, the Ace
applyToShareJS function would be overwritten by the CM version.
Also fixes bugs where adapter was calling old version of ShareJS api
and the old CM api
2018-02-09 10:38:51 +00:00
James Allen
3ac4e4c457
Run docker-compose down before acceptance tests
2018-01-19 12:42:49 +00:00
James Allen
662122bb1c
CI and local dev environment improvements
...
The need for this became very noticeable due to the slowness of filesystem access in docker-in-mac, with a full compile taking over a minute for me in docker. Using make to introduce incremental compile makes this near instantaneous outside of docker (if only a few files have changed), and quick enough inside docker.
With incremental compile via make, it compiles quickly enough that re-compiling and restarting the web service automatically when backend files change is quick enough now. This is how the service is run via docker-compose in https://github.com/sharelatex/sharelatex-dev-environment , so it shouldn't be necessary to manually restart the container each time a coffee file changes.
At the moment Jenkins pull web modules in via the GitSCM plugin, but I believe this is creating a dependency in Jenkins, where any commits to any of the modules causes all of the web branches to rebuild. By doing it via our own scripts we can hopefully avoid this. It also creates a build process which is reproducible locally.
**Note that at the moment in this PR all modules pull from `ja-dockerize-dev` branches, but these should be merged first, and this PR updated to point to the master branches before merging**. This is necessary for other changes to build process/docker-compose workflow.
As well as a Makefile for web, there is now a `Makefile.module`. This is copied into each module directory by the top-level Makefile, and is written in a way to be flexible and support unit tests, acceptance tests, front-end js for the ide and main, and the modules `app/coffee` directory, while allowing modules to have some of these missing (not all modules have e.g. acceptance tests, or front-end JS). This will allows us to refine the build process in future, without needing to update the Makefile in each module repo separately (I found this to be a painful part of this development).
This makes web compatible with the docker-compose workflow at https://github.com/sharelatex/sharelatex-dev-environment , where each service is running in its own docker container, with networking managed by docker.
Previously the Makefile was set up to run unit tests in docker with `make unit_tests`. This now just runs them natively. In the CI, they are run in docker anyway (all steps in Jenkins are), and locally, they run fine natively with `npm run test:unit`, or can be run in docker via https://github.com/sharelatex/sharelatex-dev-environment with `bin/run web_sl npm run test:unit`.
Previously we did a lot of juggling with only mounting source files (coffee, less, etc) into the docker container for acceptance tests. This was to avoid creating root owned files if the whole directory was mounted. Now instead the whole web directory is mounted read-only, with the compilation step done outside of the container before running the tests.
This allows the host and container to share the `node_modules` folder as well, which avoids needing to `npm install` twice on the CI box, and should speed up the build by a few minutes.
On macOS, this would cause a problem with compiled modules if you tried to use the same `node_modules` to run the app natively. However, if running via docker-compose in https://github.com/sharelatex/sharelatex-dev-environment , this is no longer a problem.
2018-01-15 09:36:41 +00:00
James Allen
dfe6e26946
test_unit_frontend -> test_frontend in Makefile
2017-12-19 13:58:47 +00:00
James Allen
b0812864ac
Clean up CI output
2017-12-19 13:58:47 +00:00
James Allen
d7a26e27e5
Run front end tests in CI
2017-12-19 13:58:47 +00:00
James Allen
a9ca54b98a
Generate docker-compose.yml before any docker-compose command
2017-12-02 13:02:37 +00:00
James Allen
e9733514af
Fail on failing module acceptance tests
2017-12-01 10:07:09 +00:00
James Allen
fa2a3574db
Look for Makefile, not makefile
2017-12-01 09:01:36 +00:00
James Allen
040546b1d3
Move -q flag to correct place
2017-12-01 08:49:10 +00:00
James Allen
b2a3e06717
Find / replace mistake
2017-11-30 11:20:25 +00:00
James Allen
870e87ebe1
Run npm with -q flag for less verbose test output
2017-11-30 11:00:28 +00:00
James Allen
bb74f8318a
Support make clean install
usage
2017-11-29 14:16:29 +00:00
James Allen
50b3045548
Tidy up docker-compose and makefile
2017-11-29 13:49:36 +00:00
James Allen
054964dd85
Clean out module js on make clean
2017-11-27 17:10:58 +00:00
James Allen
5e0fc24c1a
Allow modules to specific their own acceptance tests
2017-11-27 17:10:58 +00:00
James Allen
49057a5ab7
Only mount coffee and needed files into Docker so js isn't written back to local system
2017-11-27 17:10:58 +00:00