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

Protected methods #579

Open
pointlessone opened this issue Jan 18, 2021 · 3 comments
Open

Protected methods #579

pointlessone opened this issue Jan 18, 2021 · 3 comments

Comments

@pointlessone
Copy link
Contributor

RBS currently doesn't support protected method visibility. Protected methods are specified as public in generated rbs files.

Semantically protected methods are not the same as public. So I have a few questions:

  • Is this intentional?
  • Is protected visibility omitted from the initial release and will be supported later?
@castwide
Copy link
Contributor

If support for protected methods is on the roadmap, I'd be willing to submit a PR for it.

@castwide
Copy link
Contributor

Any interest in this feature? It might be uncommon, but it's still useful ("and Solargraph already supports it," I added humbly).

@skull-squadron
Copy link

Protected, private, and public all need typing checking, code-completion, and all the other benefits rbs provides.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants