Skip to content

Tests: remove tests with np.int which is a deprecated alias for buitin int#148

Merged
MuellerSeb merged 1 commit intodevelopfrom
fix_deprecation_warn_npint
Mar 26, 2021
Merged

Tests: remove tests with np.int which is a deprecated alias for buitin int#148
MuellerSeb merged 1 commit intodevelopfrom
fix_deprecation_warn_npint

Conversation

@MuellerSeb
Copy link
Copy Markdown
Member

@MuellerSeb MuellerSeb commented Mar 22, 2021

@MuellerSeb MuellerSeb added the FutureWarning Aim is to prevent FutureWarnings when they appear. label Mar 22, 2021
@MuellerSeb MuellerSeb added this to the 1.3 milestone Mar 22, 2021
@MuellerSeb MuellerSeb requested a review from LSchueler March 22, 2021 20:52
@MuellerSeb MuellerSeb self-assigned this Mar 22, 2021
@MuellerSeb
Copy link
Copy Markdown
Member Author

MuellerSeb commented Mar 22, 2021

CI currently failing for linux wheels because of this: https://status.quay.io/

@MuellerSeb
Copy link
Copy Markdown
Member Author

MuellerSeb commented Mar 22, 2021

Windows wheels still failing since #147 is not merged.

LSchueler
LSchueler approved these changes Mar 26, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@LSchueler LSchueler left a comment

Choose a reason for hiding this comment

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

I can't even remember why I tested int and np.int...

@MuellerSeb MuellerSeb merged commit ae2f672 into develop Mar 26, 2021
@MuellerSeb MuellerSeb deleted the fix_deprecation_warn_npint branch March 26, 2021 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LSchueler LSchueler LSchueler approved these changes

Assignees

@MuellerSeb MuellerSeb

Labels

FutureWarning Aim is to prevent FutureWarnings when they appear.

Projects

None yet

Milestone

1.3

Development

Successfully merging this pull request may close these issues.

2 participants

@MuellerSeb @LSchueler