Skip to content

Conversation

atolopko-czi
Copy link
Contributor

#2535

Reviewers

Functional:
@roaga

Readability:


Changes

reverted code to previous implementation that was presumably changed due to
"destructuring assignment" lint error; explicitly ignoring that error now

reverted code to previous implementation that was presumably changed due to
destructuring assignment lint error; explicitly ignoring error
@atolopko-czi atolopko-czi requested a review from roaga July 26, 2022 15:16
@atolopko-czi atolopko-czi self-assigned this Jul 26, 2022
@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #2543 (2a5ed39) into main (06da05e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2543   +/-   ##
=======================================
  Coverage   71.61%   71.61%           
=======================================
  Files          94       94           
  Lines        6496     6496           
  Branches      770      770           
=======================================
  Hits         4652     4652           
  Misses       1768     1768           
  Partials       76       76           
Flag Coverage Δ
frontend 71.61% <ø> (ø)
javascript 71.61% <ø> (ø)
smokeTest ?
unitTest 71.61% <ø> (ø)

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

Help us with your feedback. Take ten seconds to tell us how you rate us.

@atolopko-czi atolopko-czi merged commit 69a6d52 into main Jul 28, 2022
@atolopko-czi atolopko-czi deleted the atolopko/2535-fix-embedding-selection branch July 28, 2022 19:49
kathryncrouch pushed a commit to VEuPathDB/cellxgene that referenced this pull request Oct 3, 2022
reverted code to previous implementation that was presumably changed due to a "destructuring assignment" lint error; explicitly ignoring error now
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.

2 participants