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.

Sign up for GitHub

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

Jump to bottom

Remove editable annotations #162

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Remove editable annotations #162

wants to merge 6 commits into from

Conversation

ndaelman-hu
Copy link
Collaborator

Aside from removing editable quantities, this PR also removes redundant normalizers

closes #161

@ndaelman-hu ndaelman-hu added the bug Something isn't working label Jan 24, 2025
@ndaelman-hu ndaelman-hu self-assigned this Jan 24, 2025
- Remove ELNAnnotation imports
ea40f74 
- Remove 2 extra editable annotations
@coveralls
Copy link

Pull Request Test Coverage Report for Build 12952648342

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.9%) to 81.039%
Totals Coverage Status
Change from base Build 12272374018: 0.9%
Covered Lines: 2013
Relevant Lines: 2484
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JFRudzinski JFRudzinski Awaiting requested review from JFRudzinski

At least 1 approving review is required to merge this pull request.

Assignees

@ndaelman-hu ndaelman-hu

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove Str Edit Quantities from schema
2 participants
@ndaelman-hu @coveralls