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

Re roll 6279 #6341

Merged
merged 12 commits into from Oct 12, 2020
Merged

Re roll 6279 #6341

merged 12 commits into from Oct 12, 2020

Conversation

esc
Copy link
Member

@esc esc commented Oct 9, 2020

As title: re-roll #6279 due to git issues.

@esc
Copy link
Member Author

esc commented Oct 9, 2020

@stuartarchibald @sklam @rishabhvarshney14 would you be able to check if I cherry-picked everything appropriately? It's probably enough to look at the final diff of this PR.

@esc esc added the 4 - Waiting on reviewer Waiting for reviewer to respond to author label Oct 9, 2020
@esc esc added this to the Numba 0.52 RC milestone Oct 9, 2020
@@ -3906,7 +3906,11 @@ def test_asarray_chkfinite(self):
# 3x2 (with higher order broadcasting)
(
np.arange(36).reshape(6, 2, 3),
)
),
# (
Copy link
Contributor

Choose a reason for hiding this comment

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

I forgot to remove these comments here and made another commit after this to remove this

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, I think that commit was lost, I made another one to fix it, thanks!

As title.
@sklam sklam added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author labels Oct 12, 2020
@sklam sklam merged commit 01ccc3a into numba:master Oct 12, 2020
@esc
Copy link
Member Author

esc commented Oct 13, 2020

@rishabhvarshney14 finally, congrats are in order! Looks like this one made it just in the nick of time for the 0.52.0RC1 -- be on the lookout for a mention in the release notes!

@esc
Copy link
Member Author

esc commented Oct 13, 2020

@sklam thanks for approval and merge!

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