Skip to content
This repository was archived by the owner on Feb 2, 2024. It is now read-only.

Conversation

Hardcode84
Copy link
Contributor

based on numba ceab10620764d1ff29e0c0e550d1d5975674819b

@Vyacheslav-Smirnov
Copy link
Contributor

Vyacheslav-Smirnov commented Nov 28, 2019

Hi @Hardcode84 @AlexanderKalistratov ,
Unfortunately, currently CI cannot test changes for Numba master. As we decided earlier, I will create internal testing with result publishing in PR for numba-master branch.
When it will done, this PR can be merged.

@Vyacheslav-Smirnov Vyacheslav-Smirnov changed the title Merge numba master [WIP] Merge numba master Nov 28, 2019
@pep8speaks
Copy link

pep8speaks commented Dec 2, 2019

Hello @Hardcode84! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 40:36: E127 continuation line over-indented for visual indent
Line 41:36: E127 continuation line over-indented for visual indent
Line 352:30: E261 at least two spaces before inline comment

Line 42:40: E127 continuation line over-indented for visual indent
Line 43:40: E127 continuation line over-indented for visual indent

Line 39:36: E127 continuation line over-indented for visual indent
Line 40:36: E127 continuation line over-indented for visual indent

Line 37:36: E127 continuation line over-indented for visual indent
Line 38:36: E127 continuation line over-indented for visual indent

Line 36:36: E127 continuation line over-indented for visual indent
Line 37:36: E127 continuation line over-indented for visual indent

Line 45:36: E127 continuation line over-indented for visual indent
Line 46:36: E127 continuation line over-indented for visual indent

Line 38:36: E127 continuation line over-indented for visual indent
Line 39:36: E127 continuation line over-indented for visual indent

Line 41:36: E127 continuation line over-indented for visual indent
Line 42:36: E127 continuation line over-indented for visual indent

Line 37:36: E127 continuation line over-indented for visual indent
Line 38:36: E127 continuation line over-indented for visual indent

Line 37:36: E127 continuation line over-indented for visual indent
Line 38:36: E127 continuation line over-indented for visual indent

Line 4143:5: E303 too many blank lines (2)

Line 44:1: E302 expected 2 blank lines, found 1

Comment last updated at 2019-12-03 11:31:28 UTC

@Vyacheslav-Smirnov Vyacheslav-Smirnov changed the title [WIP] Merge numba master Merge numba master Dec 3, 2019
@Vyacheslav-Smirnov Vyacheslav-Smirnov merged commit b4fd225 into IntelPython:numba-master Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants