Skip to content

Conversation

@m-demare
Copy link
Contributor

@m-demare m-demare commented May 15, 2025

@jordanmontt
Copy link
Member

Great. Do you need it also merged to pharo, right?

@jordanmontt jordanmontt merged commit ea1fd39 into pharo-ai:master May 15, 2025
@jordanmontt
Copy link
Member

Because right now it will only be integrated in this repo, not in pharo. Also, you can always depend on this repo

@m-demare
Copy link
Contributor Author

Also, you can always depend on this repo

That'd make sense, now that Druid is using an algorithm from here. @PalumboN what do you say?

@jordanmontt
Copy link
Member

Depends on you. If it is integrated to pharo, it will be one less dependency in the baseline too

@PalumboN
Copy link

Is this repo integrated to the base Pharo image?

In any case, whatever that give us a green CI is ok I think :P

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