Skip to content
This repository has been archived by the owner on May 7, 2020. It is now read-only.

Move to preconfigured Resin-SDK #9

Merged
merged 1 commit into from Jan 23, 2017
Merged

Move to preconfigured Resin-SDK #9

merged 1 commit into from Jan 23, 2017

Conversation

pimterry
Copy link
Contributor

No description provided.

@jviotti
Copy link
Contributor

jviotti commented Jan 13, 2017

I approved, though I think you need to rebuild the CoffeeScript.

@pimterry
Copy link
Contributor Author

Ah, thanks. Now fixed, I'll merge once the build passes.

@Page-
Copy link

Page- commented Jan 16, 2017

So I've seen a load of these PRs pop up, and it seems that this special version of the SDK is being used everywhere? Is there ever a case when we would use the "standard" SDK? In general does it make more sense to have the "standard" SDK match the standard usage instead of the more uncommon usage, or at least have it default to the standard usage (in order to make the common case easier for everyone)?

@emirotin
Copy link

emirotin commented Jan 16, 2017 via email

@pimterry pimterry merged commit 1d03223 into master Jan 23, 2017
@pimterry pimterry deleted the preconfig-sdk branch January 23, 2017 19:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants