Skip to content

Conversation

@ksss
Copy link
Collaborator

@ksss ksss commented Mar 27, 2024

  • Should be return bot for methods of abstruct class. Use interface instead.
  • The number of arguments of Digest::Class methods cannot be determined because they are delegated to the inherited initialize.
  • Fix some types.

Good points

  • Become a more accurate signatures.

Bad points

  • Comments are effectively lost because annotations cannot be automatically written on the interface.

- Should be return bot for methods of abstruct class. Use interface instead.
- The number of arguments of Digest::Class methods cannot be determined because they are delegated to the inherited initialize.
- Fix some types.
@soutaro
Copy link
Member

soutaro commented May 29, 2024

Having -> bot types doesn't look a good idea, especially for Digest::Base. We may want to have Digest::Base type which might be one of the sub classes instances, and it all method calls will result in bot.

@ksss
Copy link
Collaborator Author

ksss commented May 29, 2024

I understand. I would like to modify it with a different approach. I will close this PR.

@ksss ksss closed this May 29, 2024
@ksss ksss deleted the digest branch May 29, 2024 12:26
@ksss ksss mentioned this pull request Jun 10, 2024
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.

2 participants