Update babel packages to match major, fix dependency issues #591

Merged

skhameneh merged 2 commits from fix-babel-deps into master

2018-09-03 17:17:36 +02:00

skhameneh commented

2018-09-03 01:51:09 +02:00

(Migrated from github.com)

Tested with yarn 1.7.0 and npm 5.6.0 (included with Node v8.11)

I'm not including a yarn.lock (yet) since we aren't using it universally.

Tested with yarn `1.7.0` and npm `5.6.0` (included with Node v8.11) I'm not including a yarn.lock (yet) since we aren't using it universally.

daovist (Migrated from github.com) approved these changes 2018-09-03 14:41:57 +02:00

daovist (Migrated from github.com) left a comment

LGTM

nikooo777 (Migrated from github.com) reviewed 2018-09-03 16:31:39 +02:00

nikooo777 (Migrated from github.com) left a comment

I noticed that we have both a yarn lockfile and a package-lock file. Are we using yarn or npm? we should remove one.

also servers should be running npm 6.4+ so I would not rely on npm 5.* anymore

I noticed that we have both a yarn lockfile and a package-lock file. Are we using yarn or npm? we should remove one. also servers should be running npm 6.4+ so I would not rely on npm 5.* anymore

skhameneh commented

2018-09-03 17:11:03 +02:00

(Migrated from github.com)

@nikooo777 the yarn.lock was added by mistake.
Why aren't the servers on LTS?

@nikooo777 the yarn.lock was added by mistake. Why aren't the servers on LTS?

No reviewers

daovist

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/spee.ch#591

No description provided.

Rows
Columns