Quasars / orange-spectroscopy

Other
52 stars 58 forks source link

[FIX] Integrate: Fix call to new deferred commit #637

Closed stuart-cls closed 1 year ago

stuart-cls commented 1 year ago

While working on #634 I found a call to commit that got missed in #622

I wasn't sure which call ("now" or "deferred") was appropriate, I chose "now"

codecov-commenter commented 1 year ago

Codecov Report

Merging #637 (fc276c8) into master (f3250d8) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #637   +/-   ##
=======================================
  Coverage   89.31%   89.31%           
=======================================
  Files          71       71           
  Lines       11817    11817           
=======================================
  Hits        10554    10554           
  Misses       1263     1263           
markotoplak commented 1 year ago

I changed to "deferred". So the current "standard" is to use deferred anywhere except for the new data received or explicit commit button clicks.