-
Notifications
You must be signed in to change notification settings - Fork 123
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
Removing edit button fix #2341
Removing edit button fix #2341
Conversation
I'm highly skeptical of this honestly |
Codecov Report
@@ Coverage Diff @@
## main #2341 +/- ##
==========================================
- Coverage 87.67% 87.42% -0.25%
==========================================
Files 45 44 -1
Lines 8527 8455 -72
==========================================
- Hits 7476 7392 -84
- Misses 1051 1063 +12 |
Waiting for a new release of |
I'm going to merge "blindly" because I cannot check without it getting online I guess????..... |
@Revathyvenugopal162 you ok with that? |
It seems to work ok. I am checking the changes. For example this link:
I see a bit of redundancy, but I would probably keep both. However I would expect that the editor mode will take you to the right lines. But maybe it is not possible. |
|
It should remove redundancy in code.
Close #2254