Skip to content

Test data changes for dependency update #77

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.

Sign up for GitHub

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

Jump to bottom
Merged
bjlittle merged 2 commits into from
Jun 27, 2022
Merged

Test data changes for dependency update #77

bjlittle merged 2 commits into from
Jun 27, 2022

Conversation

@wjbenfold
Copy link
Contributor

@wjbenfold wjbenfold commented Jun 24, 2022

No description provided.

@wjbenfold wjbenfold mentioned this pull request Jun 24, 2022
Merged
@wjbenfold wjbenfold requested a review from bjlittle June 24, 2022 10:52
@rcomer
Copy link
Member

rcomer commented Jun 24, 2022

This new system makes it so much easier to see what's actually changing 🤩

@wjbenfold
Copy link
Contributor Author

wjbenfold commented Jun 24, 2022

This new system makes it so much easier to see what's actually changing 🤩

Maybe idiff.py's days are numbered, given you can use this to spot what's changing...

@wjbenfold wjbenfold mentioned this pull request Jun 24, 2022
Merged
@bjlittle bjlittle merged commit 3b583a2 into SciTools:master Jun 27, 2022
@bjlittle
Copy link
Member

bjlittle commented Jun 27, 2022

Maybe idiff.py's days are numbered, given you can use this to spot what's changing...

I think that idiff still has value for more subtle changes, as it easily highlights pixel level changes... but, yeah, I know what you mean 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bjlittle bjlittle bjlittle approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wjbenfold @rcomer @bjlittle