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

Document string support #3506

Merged
merged 6 commits into from Nov 19, 2018
Merged

Document string support #3506

merged 6 commits into from Nov 19, 2018

Conversation

seibert
Copy link
Contributor

@seibert seibert commented Nov 16, 2018

Documentation for preliminary string support

@codecov-io
Copy link

codecov-io commented Nov 17, 2018

Codecov Report

Merging #3506 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3506      +/-   ##
==========================================
+ Coverage   80.62%   80.69%   +0.06%     
==========================================
  Files         392      392              
  Lines       79770    80130     +360     
  Branches     9063     9117      +54     
==========================================
+ Hits        64317    64663     +346     
- Misses      14038    14051      +13     
- Partials     1415     1416       +1

Copy link
Contributor

@stuartarchibald stuartarchibald left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, looks good, two minor comments as edits for consistent use of nopython mode otherwise can be merged.

docs/source/reference/pysupported.rst Outdated Show resolved Hide resolved
docs/source/reference/pysupported.rst Outdated Show resolved Hide resolved
stuartarchibald and others added 2 commits November 19, 2018 08:51
Co-Authored-By: seibert <stan@mtrr.org>
Co-Authored-By: seibert <stan@mtrr.org>
@seibert
Copy link
Contributor Author

seibert commented Nov 19, 2018

OK, I have accepted the proposed change.

@seibert seibert added this to the Numba 0.41 RC milestone Nov 19, 2018
@seibert seibert added 4 - Waiting on reviewer Waiting for reviewer to respond to author and removed 3 - Ready for Review labels Nov 19, 2018
@stuartarchibald
Copy link
Contributor

hmmm, that edit functionality on github doesn't line wrap or deal with white space, guess that's expected, so apologies, this needs manual fixing now. The change renders ok as HTML (apart from a space before the full stop), but the source lines overrun. Perhaps edit manually?! Guess we've learned not to do that again for RST.

@stuartarchibald stuartarchibald added 4 - Waiting on author Waiting for author to respond to review and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author labels Nov 19, 2018
@seibert seibert added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on author Waiting for author to respond to review labels Nov 19, 2018
@seibert seibert merged commit f12b3fb into numba:master Nov 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Ready to merge Review and testing done, is ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants