Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trim down endpoints to limited endpoint set #44

Merged
merged 1 commit into from
Apr 21, 2018

Conversation

emrebsonmez
Copy link
Contributor

This removes all endpoints from the Smartcar Node SDK and limits our endpoint set to: info, vin, permissions, disconnect, odometer, location, lock, and unlock.

BREAKING CHANGE: This removes all endpoints from the Smartcar Node SDK and limits our endpoint set to: info, vin, permissions, disconnect, odometer, location, lock, and unlock.
Copy link
Contributor

@GregChan GregChan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you run the docs npm script to limit the endpoints for index.md?

Also if you squash and merge, please make sure your commit message matches semantic release format.

Copy link
Contributor

@sankethkatta sankethkatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we not need to edit the package.json version? @gurpreetatwal @GregChan

Also will this auto-deploy to npm upon merge?

@emrebsonmez
Copy link
Contributor Author

@GregChan ran npm run docs. Please let me know if I can be of further assistance 🙏

@gurpreetatwal
Copy link
Contributor

@sankethkatta no need to touch package.json version anymore, semantic release will compute the correct version and then create a git tag and publish to npm

This would be version 2.0.0 since it's a breaking change from 1.0.0

@emrebsonmez emrebsonmez merged commit 0171b5d into master Apr 21, 2018
@emrebsonmez emrebsonmez deleted the condense-endpoints branch April 21, 2018 01:51
@emrebsonmez
Copy link
Contributor Author

@gurpreetatwal was this supposed to be perf()?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants