Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
for FieldDB/FieldDB#2173
Before Node 8 and Node 10: https://travis-ci.org/github/FieldDB/dative/jobs/708989910
After Node 12 : https://travis-ci.org/github/FieldDB/dative/jobs/708989911
List of impacted files:
Root cause appears that all of these files have nested
<%
when i remove it or workaround it, the file is able to compile:I think i found that i can reproduce this in eco's test suites using 1.0.3 but using the latest version v1.1.0-rc-3 works so upgrading to the latest might be a direction we can go in but it requires an update to grunt-eco to handle the breaking change where compile now returns a function
Option 1: fix the compile issue
coffeescript
instead ofcoffee-script
https://github.com/caolan/nodeunit/pull/358/filesOption 2: check in the dist
If we cant build on the server, we can build using node 10 and then check in the dist and serve those using node 12 FieldDB#16
Option 3: convert from coffee to js
There is probably a util that we can use to convert all the coffee files into es6. i did a POC using decaffeinate but it would require more work to update the gruntfile to use eslint and another templating engine that is similar to eco FieldDB#17