Skip to content

Conversation

@nobu
Copy link
Member

@nobu nobu commented Sep 6, 2021

No description provided.

@nobu nobu requested a review from knu September 6, 2021 04:12
@nobu nobu force-pushed the move-digest.rb-back branch from 1f26e14 to 026ba7f Compare September 7, 2021 04:44
@nobu nobu merged commit d39db59 into ruby:master Sep 7, 2021
@nobu nobu deleted the move-digest.rb-back branch September 7, 2021 05:00
@knu
Copy link
Member

knu commented Sep 7, 2021

You should have waited for #20 before doing this because it causes a conflict.

@hsbt
Copy link
Member

hsbt commented Sep 7, 2021

And I'm not sure why we leave from the standard gem layout with this changes.

@knu
Copy link
Member

knu commented Sep 7, 2021

My plan is to have platform independent parts in lib/digest.rb and platform specific parts in ext, one for CRuby and another for JRuby.
#21 (comment)

@hsbt
Copy link
Member

hsbt commented Sep 7, 2021

@knu I agreed that.

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.

3 participants