Permalink
Browse files

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.
  • Loading branch information...
1 parent 9302a27 commit 632ffdfa8f5c0a13d411967d045ff1f46a5df454 @calzoneman committed Nov 3, 2016
Showing with 23 additions and 3 deletions.
  1. +20 −0 NEWS.md
  2. +1 −1 config.template.yaml
  3. +2 −2 package.json
View
@@ -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
==========
@@ -91,7 +91,7 @@ https:
# Page template values
# title goes in the upper left corner, description goes in a <meta> tag
html-template:
- title: 'Sync',
+ title: 'Sync'
description: 'Free, open source synchtube'
# Socket.IO server details
View
@@ -2,7 +2,7 @@
"author": "Calvin Montgomery",
"name": "CyTube",
"description": "Online media synchronizer and chat",
- "version": "3.23.6",
+ "version": "3.24.0",
"repository": {
"url": "http://github.com/calzoneman/sync"
},
@@ -46,7 +46,7 @@
"status-message-polyfill": "git://github.com/calzoneman/status-message-polyfill",
"toml": "^2.3.0",
"uuid": "^2.0.1",
- "yamljs": "^0.1.6"
+ "yamljs": "^0.2.8"
},
"scripts": {
"build-player": "$npm_node_execpath build-player.js",

0 comments on commit 632ffdf

Please sign in to comment.