Upgrade feature Intended for

[arnaudbore]

No description provided.

[codecov-commenter]

Codecov Report

Merging #207 (9e8eb90) into master (803848e) will increase coverage by 0.77%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
+ Coverage   78.00%   78.78%   +0.77%     
==========================================
  Files          15       15              
  Lines         773      773              
  Branches      109      111       +2     
==========================================
+ Hits          603      609       +6     
+ Misses        138      134       -4     
+ Partials       32       30       -2     

Flag	Coverage Δ
pytest	78.78% <96.29%> (+0.77%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
dcm2bids/cli/dcm2bids.py	84.00% <ø> (+5.42%)	⬆️
dcm2bids/sidecar.py	87.50% <ø> (-0.09%)	⬇️
dcm2bids/dcm2bids_gen.py	82.56% <87.50%> (+3.47%)	⬆️
dcm2bids/acquisition.py	98.24% <100.00%> (+0.08%)	⬆️
dcm2bids/participant.py	100.00% <100.00%> (ø)
dcm2bids/utils/utils.py	76.47% <100.00%> (+0.47%)	⬆️

[SamGuay]

id field works great but I wonder whether we should keep the former method as well to prevent current pipeline from breaking as all config files would need to be updated. So maybe printing a warning instead of an error?