Skip to content

Changed wording of the escape error - #8782

Merged
sklam merged 1 commit into
numba:mainfrom
MiloniAtal:Escape
Feb 25, 2023
Merged

sklam merged 1 commit into
numba:mainfrom
MiloniAtal:Escape

Conversation

@MiloniAtal

Copy link
Copy Markdown
Contributor

Resolves #7254
Changed the wording of the error according to the comments in the issue #7254.

No tests failed when the wording was changed.

@kc611 kc611 added 3 - Ready for Review doc CUDA CUDA related issue/PR and removed CUDA CUDA related issue/PR labels Feb 24, 2023

@stuartarchibald stuartarchibald left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the patch @MiloniAtal.

@stuartarchibald stuartarchibald added 5 - Ready to merge Review and testing done, is ready to merge Effort - short Short size effort needed and removed 3 - Ready for Review labels Feb 24, 2023
@stuartarchibald stuartarchibald added this to the Numba 0.57 RC milestone Feb 24, 2023
@stuartarchibald stuartarchibald self-assigned this Feb 24, 2023
@sklam
sklam merged commit 720b357 into numba:main Feb 25, 2023
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 doc Effort - short Short size effort needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unused closures in njitted function are wrongly reported as escaping

4 participants