Skip to content

Refine Digest - #1779

Closed
ksss wants to merge 1 commit into
ruby:masterfrom
ksss:digest
Closed

Refine Digest#1779
ksss wants to merge 1 commit into
ruby:masterfrom
ksss:digest

Conversation

@ksss

@ksss ksss commented Mar 27, 2024

Copy link
Copy Markdown
Collaborator
  • 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

soutaro commented May 29, 2024

Copy link
Copy Markdown
Member

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

ksss commented May 29, 2024

Copy link
Copy Markdown
Collaborator Author

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