ruby / prism

Prism Ruby parser
https://ruby.github.io/prism/
MIT License
845 stars 142 forks source link

Bump Chicory to 1.0.0-M1 #3210

Closed andreaTP closed 2 weeks ago

andreaTP commented 2 weeks ago

:wave: we just released a first Milestone for Chicory, since it's breaking in many ways, I wanted to check that everything remains aligned.

Notes:

kddnewton commented 2 weeks ago

Yeah that sounds great! Do you mean add it as an artifact after the build kind of thing? @enebo would you be able to use that?

andreaTP commented 2 weeks ago

I think @enebo will be able to use it easily, I would require some help in setting up the automated release CI,. If that's good for you @kddnewton I'll move forward in the next few days. 😊

BTW, a wonderful experience contributing to this repo! Great job!

kddnewton commented 2 weeks ago

Yeah that's fine. I think just uploading it as an artifact to the build to start would be good. That's what we have for JS and it works fine for now https://github.com/ruby/prism/blob/efd3362a91f9fef296e42e931ef637e60cea18d1/.github/workflows/javascript-bindings.yml#L37-L40

andreaTP commented 2 weeks ago

Thanks for the feedback, @kddnewton , two comments here:

All in all, the current setup is fine for now, I'll come up with something when we are ready :slightly_smiling_face:

kddnewton commented 2 weeks ago

Sounds great!