-
Notifications
You must be signed in to change notification settings - Fork 922
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
Revert MethodDeclarationId change due to breaking code. #1615
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1615 +/- ##
==========================================
- Coverage 53.42% 53.42% -0.01%
==========================================
Files 312 312
Lines 57367 57368 +1
==========================================
Hits 30649 30649
- Misses 26718 26719 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds like we need the - non-RC version of the nodeset
right now branch for testing only... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HasTypeDefinition reference has been eliminated from the method instance as expected.
Reverts the changes. Need to see if removing the Reference solves the original problem.