Skip to content

Conversation

bprize15
Copy link

@bprize15 bprize15 changed the base branch from master to rc July 16, 2025 16:46
@bprize15 bprize15 requested a review from calvinlu3 July 16, 2025 16:46
Copy link
Contributor

@calvinlu3 calvinlu3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

alterationWithEntityStatus.getEntity().setProteinChange(proteinChangeOptional.orElseThrow());

String hgvsg = variantAnnotation.getHgvsg();
if (hgvsg == null) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to be safe, check hgvsg.isEmpty()?

@bprize15 bprize15 merged commit 79f019b into oncokb:rc Jul 22, 2025
2 checks passed
@bprize15 bprize15 deleted the feat/support-hgvsg branch July 22, 2025 14:23
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