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

Api stubs #1208

Merged
merged 2 commits into from May 19, 2016

Conversation

Projects
None yet
2 participants
@richafrank
Member

richafrank commented May 17, 2016

as well as tooling and docs to generate this for each release

For #1205

screen shot 2016-05-17 at 10 34 42 am

def main():
with open(api.__file__ + 'i', 'w') as stub:

This comment has been minimized.

@llllllllll

llllllllll May 19, 2016

Member

we should rstrip 'c' from the filename because it could be .pyc

This comment has been minimized.

@richafrank

richafrank May 19, 2016

Member

Agreed.

@@ -27,6 +27,21 @@ update the underline of the title to match the title's width.
If you are renaming the release at this point, you'll need to git mv the file
and also update releases.rst to reference the renamed file.
Updating the Python stub files

This comment has been minimized.

@llllllllll

llllllllll May 19, 2016

Member

should we comment that the imports may need to be updated?

This comment has been minimized.

@richafrank

richafrank May 19, 2016

Member

Good idea.

@richafrank richafrank force-pushed the api_stubs branch from cf00fef to 7a7b5db May 19, 2016

richafrank added some commits May 16, 2016

DEV: Adds type hinting stub for zipline.api
as well as tooling and docs to generate this for each release

Also moved Cython files to package_data, so that we install them,
instead of just packaging them.

@richafrank richafrank force-pushed the api_stubs branch from 7a7b5db to 7afd486 May 19, 2016

@richafrank richafrank merged commit 1d16309 into master May 19, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@richafrank richafrank deleted the api_stubs branch May 19, 2016

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