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

Change default access modifier to internal #69

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

arg0d
Copy link
Contributor

@arg0d arg0d commented Feb 8, 2024

Access modifier may be configured via access_modifier configuration value.

Closes #23

@arg0d arg0d requested a review from Lipt0nas February 8, 2024 13:58
@arg0d arg0d force-pushed the kristupas/change-default-visibility-modifier branch from 5703ea8 to 3028855 Compare February 8, 2024 13:58
@arg0d
Copy link
Contributor Author

arg0d commented Feb 8, 2024

@AArnott

Access modifier may be configured via `access_modifier`
configuration value.

Signed-off-by: Kristupas Antanavičius <kristupas.antanavicius@nordsec.com>
@arg0d arg0d force-pushed the kristupas/change-default-visibility-modifier branch from 3028855 to 1e6c86c Compare February 8, 2024 14:36
@arg0d arg0d changed the title Change default visibility modifier to internal Change default access modifier to internal Feb 8, 2024
Copy link
Contributor

@AArnott AArnott left a comment

Choose a reason for hiding this comment

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

Love it! Thanks.

@arg0d arg0d merged commit 861f4cb into main Feb 9, 2024
3 checks passed
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.

Option to generate internal APIs
2 participants