Skip to content

ENH: Simplify Nifti IO test 13 using ReadImage and testing macros#3448

Merged
dzenanz merged 1 commit into
InsightSoftwareConsortium:masterfrom
dzenanz:niftiTest13
May 25, 2022
Merged

ENH: Simplify Nifti IO test 13 using ReadImage and testing macros#3448
dzenanz merged 1 commit into
InsightSoftwareConsortium:masterfrom
dzenanz:niftiTest13

Conversation

@dzenanz
Copy link
Copy Markdown
Member

@dzenanz dzenanz commented May 25, 2022

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)

@dzenanz dzenanz mentioned this pull request May 25, 2022
Merged
5 tasks
@github-actions github-actions Bot added area:IO Issues affecting the IO module type:Enhancement Improvement of existing methods or implementation type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct labels May 25, 2022
@tbirdso tbirdso mentioned this pull request May 25, 2022
Merged
1 task
@tbirdso
Copy link
Copy Markdown
Contributor

tbirdso commented May 25, 2022

Please rebase on master to address the ITK.Linux CI failure.

@dzenanz @N-Dekker
ENH: Simplify Nifti IO test 13 using ReadImage and testing macros
ee664da 
Co-authored-by: Niels Dekker <n.dekker@lumc.nl>
@dzenanz dzenanz merged commit ee664da into InsightSoftwareConsortium:master May 25, 2022
@dzenanz dzenanz deleted the niftiTest13 branch May 25, 2022 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gdevenyi gdevenyi Awaiting requested review from gdevenyi

@N-Dekker N-Dekker Awaiting requested review from N-Dekker

@jhlegarreta jhlegarreta Awaiting requested review from jhlegarreta

Assignees

No one assigned

Labels

area:IO Issues affecting the IO module type:Enhancement Improvement of existing methods or implementation type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dzenanz @tbirdso