Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
48 changed files
with
98 additions
and
96 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,8 @@ | ||
# TeslaJS Change log | ||
|
||
## V4.0.0 | ||
* migrated to all lower-case | ||
|
||
## V3.0.0 | ||
* addressed #47 | ||
|
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,36 @@ | ||
# TeslaJS Migration Buid | ||
|
||
## V2.x to V3.0.0 | ||
## V3.x to V4.x | ||
|
||
Unfortunately we learned recently that npm does not respect case-sensitive file systems. | ||
As a result on Mac and Linux if you: | ||
|
||
npm install teslajs | ||
|
||
as the library documentation suggests, npm will install the library under node_modules | ||
into a directory called `teslajs`. This will break calls to `require("TeslaJS")`. | ||
However, if you instead: | ||
|
||
npm install TeslaJS | ||
|
||
npm will install the library under node_modules into a directory called `TeslaJS`. | ||
This will work correctly with `require("TelsaJS")`. It seems the right solution is | ||
to attempt to remove case issues from the equation. | ||
|
||
Unfortunately the rules of `semver` say this is a breaking change, thus the bump | ||
from 3.x to 4.x. So the **big** change from 3.x to 4.x is simply to update all `require` | ||
statements to use all lower-case as in: | ||
|
||
npm install teslajs | ||
... | ||
require('teslajs'); | ||
|
||
Since I was removing case from the equation I also renamed the `Examples` dir to | ||
be `samples` instead. | ||
|
||
Apologies for the churn! | ||
|
||
## V2.x to V3.x | ||
|
||
The primary breaking change in V3.x compared to 2.x is the API renaming change of | ||
`allVehicles()` to `vehicles()` and `vehicles()` to `vehicle()`. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
@echo off | ||
|
||
echo========================== Generating Docs ================================ | ||
cmd /c ".\node_modules\.bin\jsdoc2md" TeslaJS.js > docs\DOCS.md | ||
cmd /c ".\node_modules\.bin\jsdoc2md" teslajs.js > docs\DOCS.md | ||
echo ..Documentation generated.. | ||
echo=========================================================================== |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Oops, something went wrong.