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

remove @internal annotation #1514

Merged
merged 1 commit into from Dec 10, 2020
Merged

remove @internal annotation #1514

merged 1 commit into from Dec 10, 2020

Conversation

idevin
Copy link
Contributor

@idevin idevin commented Nov 19, 2020

getClass is not internal method

getClass is not internal method
@stof
Copy link
Contributor

stof commented Nov 19, 2020

Well, why not ?

@Seldaek Seldaek changed the base branch from master to main December 9, 2020 22:34
@Seldaek
Copy link
Owner

Seldaek commented Dec 10, 2020

Eh well, I guess it's harmless to allow third parties to use this. It was internal initially because it was added on the Registry class where it really did not make a ton of sense, but now on the Utils class it's unlikely to ever go away.

@Seldaek Seldaek merged commit 7a2ee8d into Seldaek:main Dec 10, 2020
@Seldaek Seldaek added this to the 2.x milestone Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants