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

Enforced explicit API mode #10

Closed
wants to merge 2 commits into from
Closed

Enforced explicit API mode #10

wants to merge 2 commits into from

Conversation

zTrap
Copy link

@zTrap zTrap commented Apr 4, 2023

Fixes #9

Includes changes from #8

@zTrap zTrap requested a review from sksamuel as a code owner April 4, 2023 12:55
@sksamuel
Copy link
Owner

I like this change, can you make the PR independent of #8 because I'm not sold on changing the API to introduce interfaces.

@zTrap zTrap closed this Feb 12, 2024
@zTrap zTrap deleted the feature/enforce-explicit-api-mode branch February 12, 2024 14:27
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.

Enforce explicit API visibility modifiers
2 participants