search

cronofy / cronofy-node

Node wrapper for the Cronofy API
https://docs.cronofy.com/developers
MIT License
49 stars 23 forks source link

remove const in favor of var #33

Closed warbrett closed 7 years ago

warbrett commented 7 years ago

With babel removed I believe this is the last change necessary to ensure this runs on ancient versions of Node still.

gshutler commented 7 years ago

Thanks! Released in 2.3.2

AdriVanHoudt commented 7 years ago

Missed that haha

What is the lowest supported version? You can add a key to package.json to state this. e.g.

  "engines": {
    "node": ">=4.8.2"
  },
stephenbinns commented 7 years ago

Does this also mean we can remove babel-cli and babel-preset-es2015 from the devDepencencies?

AdriVanHoudt commented 7 years ago

ah yeah totally, kinda rushed the fix sorry!