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.

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

Check visibility of methods in Plan #1086

Open
qinsoon opened this issue Feb 21, 2024 · 0 comments
Open

Check visibility of methods in Plan #1086

qinsoon opened this issue Feb 21, 2024 · 0 comments
Labels
A-interface Area: Interface/API C-bug Category: Bug P-high Priority: High. A high-priority issue should be fixed as soon as possible.

Comments

@qinsoon
Copy link
Member

qinsoon commented Feb 21, 2024

Currently Plan is a public trait, and all the methods in Plan are public. However, many methods in Plan should not be public. We can either split Plan into two traits, one with public visibility and one with private visibility, or we can introduce a public type that wraps Plan (like SpaceStats for Space in #1083).
@qinsoon qinsoon added C-bug Category: Bug A-interface Area: Interface/API P-high Priority: High. A high-priority issue should be fixed as soon as possible. labels Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-interface Area: Interface/API C-bug Category: Bug P-high Priority: High. A high-priority issue should be fixed as soon as possible.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@qinsoon