Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keep old reference in the state if they have not changed #2114

Merged
merged 5 commits into from
Jul 29, 2022

Conversation

sroy3
Copy link
Contributor

@sroy3 sroy3 commented Jul 28, 2022

Small change. I don't think it'll boost performance by that much since we're always passing the whole data, but it certainly does not hurt and was easy to put in.

@sroy3 sroy3 self-assigned this Jul 28, 2022
@sroy3 sroy3 marked this pull request as ready for review July 28, 2022 17:36
@sroy3 sroy3 enabled auto-merge (squash) July 29, 2022 12:58
@codeclimate
Copy link

codeclimate bot commented Jul 29, 2022

Code Climate has analyzed commit aa108ef and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.7% (0.0% change).

View more on Code Climate.

@sroy3 sroy3 merged commit ed4c976 into main Jul 29, 2022
@sroy3 sroy3 deleted the keep-old-references-tableData branch July 29, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants