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.

Sign up for GitHub

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

Jump to bottom

Don't include private_constant-ed module #155

Merged

kou merged 1 commit into ruby:master from naitoh:fix_private_constant

Jun 22, 2024

Contributor

naitoh commented Jun 22, 2024 •

edited by kou

Loading

Included constants are not private. So private constants in private module aren't private.

See also: #154 (comment)


          Fix private_constant being public

c8e69c3

See: ruby#154 (comment)

kou changed the title ~~Fix private_constant being public~~ Don't include private_constant-ed module

kou merged commit cfa8dd9 into ruby:master

61 checks passed

Member

kou commented Jun 22, 2024

Thanks.

naitoh deleted the fix_private_constant branch

June 22, 2024 05:31

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