Skip to content

Conversation

@BenMusch
Copy link
Member

Too many people are gonna ping the server for pairings, etc. For this to be viable

Instead we should just wrap important logic in transactions

@codecov-io
Copy link

codecov-io commented Nov 22, 2019

Codecov Report

Merging #281 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
- Coverage   61.34%   61.33%   -0.01%     
==========================================
  Files          63       63              
  Lines        4237     4236       -1     
==========================================
- Hits         2599     2598       -1     
  Misses       1638     1638
Impacted Files Coverage Δ
mittab/settings.py 95.83% <ø> (-0.17%) ⬇️

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 167b4fd...9e2805e. Read the comment docs.

@BenMusch BenMusch merged commit daa91af into master Nov 22, 2019
roddajohn pushed a commit to roddajohn/mit-tab that referenced this pull request Feb 26, 2023
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.

3 participants