Skip to content

Conversation

ghisvail
Copy link
Collaborator

The attrs namespace is now preferred over the old attr. It has been introduced since version 21.3.0, hence the necessary minimum version specified in the package metadata.

@ghisvail
Copy link
Collaborator Author

This is a follow-up to Omar's PR (#22). The attrs namespace provides more sensible defaults for asdict.

@codecov-commenter
Copy link

codecov-commenter commented Nov 21, 2022

Codecov Report

Base: 95.83% // Head: 95.83% // No change to project coverage 👍

Coverage data is based on head (57b4fd4) compared to base (75b72f2).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage   95.83%   95.83%           
=======================================
  Files           2        2           
  Lines          24       24           
  Branches        3        3           
=======================================
  Hits           23       23           
  Partials        1        1           
Impacted Files Coverage Δ
pydra/tasks/nipype1/utils.py 95.23% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ghisvail ghisvail force-pushed the mnt/use-attrs-namespace branch from 96eb350 to 57b4fd4 Compare November 21, 2022 18:43
@ghisvail ghisvail merged commit 71e6b59 into nipype:main Nov 21, 2022
@ghisvail ghisvail deleted the mnt/use-attrs-namespace branch November 21, 2022 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants