hmpf / easydmp

MIT License
7 stars 2 forks source link

Add a repeatable flag to Section #172

Closed vbhavdal closed 3 years ago

vbhavdal commented 3 years ago

Is this a sensible first step?

codecov[bot] commented 3 years ago

Codecov Report

Merging #172 (64a65d9) into master (6242b07) will increase coverage by 0.03%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
+ Coverage   55.15%   55.19%   +0.03%     
==========================================
  Files         102      103       +1     
  Lines        5903     5908       +5     
==========================================
+ Hits         3256     3261       +5     
  Misses       2647     2647              
Impacted Files Coverage Δ
...easydmp/dmpt/migrations/0012_section_repeatable.py 100.00% <100.00%> (ø)
src/easydmp/dmpt/models.py 52.97% <100.00%> (+0.03%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6242b07...64a65d9. Read the comment docs.

hmpf commented 3 years ago

Is this a sensible first step?

Absolutely. When that one is in we can argue about where to store answers of repeated sections :)

vbhavdal commented 3 years ago

Redo after squash