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.

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

Make ExtendedNode and Label public again #6130

Merged

Conversation

msridhar
Copy link
Contributor

In #5955, the ExtendedNode and Label classes from the dataflow framework were changed from public to package-private. However, NullAway relies on these classes being public (see #5156, #5152, and uber/NullAway#608). This PR makes the classes public again and adds an explanatory comment.

@msridhar msridhar requested a review from mernst August 12, 2023 22:56
@msridhar msridhar enabled auto-merge (squash) August 12, 2023 22:57
Copy link
Member

@mernst mernst left a comment

Choose a reason for hiding this comment

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

Thanks!

@msridhar msridhar merged commit 9bff963 into typetools:master Aug 14, 2023
29 checks passed
@msridhar msridhar deleted the make-extended-node-and-label-public branch August 14, 2023 06:28
wmdietl pushed a commit to eisop/checker-framework that referenced this pull request Oct 16, 2023
In typetools#5955, the `ExtendedNode` and `Label` classes from the dataflow
framework were changed from public to package-private. However, NullAway
relies on these classes being public (see
typetools#5156,
typetools#5152, and
uber/NullAway#608). This PR makes the classes
public again and adds an explanatory comment.
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.

None yet

2 participants