Skip to content

Conversation

dmlittle
Copy link
Contributor

@dmlittle dmlittle commented Jun 5, 2015

@elnaz @leore @pon
updated python wrapper examples. renamed folders structures and flushed out the readme

Copy link
Contributor

Choose a reason for hiding this comment

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

I think the example should point to the master copy of Lob. Not the package version.

Copy link
Contributor

Choose a reason for hiding this comment

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

It is pointing to the master copy. I was wrong yesterday.
They do still need to install the package once to use 'import lob'.

Copy link
Contributor

Choose a reason for hiding this comment

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

So what he's saying is that we should be referencing the local code rather than importing lob from pip. This prevents any issues with their installed version differing from the downloaded version. So the examples will always work.

@pon
Copy link
Contributor

pon commented Jun 5, 2015

LGTM

1 similar comment
@mgartner
Copy link
Contributor

mgartner commented Jun 5, 2015

LGTM

@dmlittle dmlittle force-pushed the improve_examples_onboarding branch from 375e344 to ef610d7 Compare June 5, 2015 22:51
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@leore @pon
This looks kinda hacky but it will load the unpacked version before anything else. Is there a better way of doing it? It's completely dependent on the relative path of the file.

@dmlittle dmlittle force-pushed the improve_examples_onboarding branch from ef610d7 to 350597a Compare June 6, 2015 22:21
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 350597a on improve_examples_onboarding into 7dfbfd1 on master.

mgartner added a commit that referenced this pull request Jun 9, 2015
feat(examples): making easier to follow examples
@mgartner mgartner merged commit 3caa2e9 into master Jun 9, 2015
@mgartner mgartner deleted the improve_examples_onboarding branch June 9, 2015 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

6 participants