mirror of https://github.com/calzoneman/sync.git
deps: upgrade yamljs
Old version of `yamljs` was bringing in deprecated dependencies causing `npm install` warnings. Newer version is still buggy and doesn't fully support the YAML spec, but it seems to work at least as well as the old version, as far as I can tell.
This commit is contained in:
parent
9302a271d0
commit
632ffdfa8f
20
NEWS.md
20
NEWS.md
|
@ -1,3 +1,23 @@
|
||||||
|
2016-11-02
|
||||||
|
==========
|
||||||
|
|
||||||
|
After upgrading the dependency on `yamljs`, you may see this error if you didn't
|
||||||
|
notice and correct a typo in the config.yaml template:
|
||||||
|
|
||||||
|
Error loading config file config.yaml:
|
||||||
|
{ [Error: Unexpected characters near ",".]
|
||||||
|
message: 'Unexpected characters near ",".',
|
||||||
|
parsedLine: 88,
|
||||||
|
snippet: 'title: \'CyTube\',' }
|
||||||
|
|
||||||
|
The fix is to edit config.yaml and remove the trailing comma for the `title:`
|
||||||
|
property under `html-template`. If there are other syntax errors that the old
|
||||||
|
version didn't detect, you will need to correct those as well.
|
||||||
|
|
||||||
|
Longer term, I am looking to move away from using `yamljs` to parse
|
||||||
|
configuration because it's a little buggy and the current configuration system
|
||||||
|
is confusing.
|
||||||
|
|
||||||
2016-10-20
|
2016-10-20
|
||||||
==========
|
==========
|
||||||
|
|
||||||
|
|
|
@ -91,7 +91,7 @@ https:
|
||||||
# Page template values
|
# Page template values
|
||||||
# title goes in the upper left corner, description goes in a <meta> tag
|
# title goes in the upper left corner, description goes in a <meta> tag
|
||||||
html-template:
|
html-template:
|
||||||
title: 'Sync',
|
title: 'Sync'
|
||||||
description: 'Free, open source synchtube'
|
description: 'Free, open source synchtube'
|
||||||
|
|
||||||
# Socket.IO server details
|
# Socket.IO server details
|
||||||
|
|
|
@ -2,7 +2,7 @@
|
||||||
"author": "Calvin Montgomery",
|
"author": "Calvin Montgomery",
|
||||||
"name": "CyTube",
|
"name": "CyTube",
|
||||||
"description": "Online media synchronizer and chat",
|
"description": "Online media synchronizer and chat",
|
||||||
"version": "3.23.6",
|
"version": "3.24.0",
|
||||||
"repository": {
|
"repository": {
|
||||||
"url": "http://github.com/calzoneman/sync"
|
"url": "http://github.com/calzoneman/sync"
|
||||||
},
|
},
|
||||||
|
@ -46,7 +46,7 @@
|
||||||
"status-message-polyfill": "git://github.com/calzoneman/status-message-polyfill",
|
"status-message-polyfill": "git://github.com/calzoneman/status-message-polyfill",
|
||||||
"toml": "^2.3.0",
|
"toml": "^2.3.0",
|
||||||
"uuid": "^2.0.1",
|
"uuid": "^2.0.1",
|
||||||
"yamljs": "^0.1.6"
|
"yamljs": "^0.2.8"
|
||||||
},
|
},
|
||||||
"scripts": {
|
"scripts": {
|
||||||
"build-player": "$npm_node_execpath build-player.js",
|
"build-player": "$npm_node_execpath build-player.js",
|
||||||
|
|
Loading…
Reference in New Issue