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

fix https://github.com/yegor256/takes/issues/945 #1370

Closed
wants to merge 1 commit into from

Conversation

Kartofanych
Copy link

@Kartofanych Kartofanych commented Dec 14, 2024

move PqBaseMethod outside of parent interface
fix #945

@Kartofanych Kartofanych force-pushed the move_rq_base_method branch 2 times, most recently from f047e75 to a47374f Compare December 14, 2024 20:44
@Kartofanych
Copy link
Author

@yegor256 check please

@yegor256
Copy link
Owner

@Kartofanych what is the motivation for this? What problem are you trying to fix with these changes?

@Kartofanych
Copy link
Author

Kartofanych commented Dec 17, 2024

@Kartofanych what is the motivation for this? What problem are you trying to fix with these changes?

this is fix of issue #945
you asked to check issues that repository has - i found this one
problem is that interface contains its implementations inside - its not good practice according to issue

@Kartofanych
Copy link
Author

@yegor256 Any updates on this? (course is already finished)

@yegor256
Copy link
Owner

@Kartofanych the issue #945 was just a proposal, not an accepted bug (I closed it now). Please, look at the issues that are labeled as "bug" only

@Kartofanych Kartofanych deleted the move_rq_base_method branch December 27, 2024 07:53
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.

Move all classes inside interfaces to outside
2 participants