Update installation script

[drkameleon]

Minor changes to align the installation script with current naming conventions in our Nightly releases.

See also: https://discord.com/channels/765519132186640445/1285475027801739307/1293811512737140839

---

Obviously, another workaround could be directly using the installation script:

curl -sSL https://get.arturo-lang.io | sh -s -- --nightly

[drkameleon]

Related: #97

[ErikSchierboom]

@drkameleon CI now fails: https://github.com/exercism/arturo/actions/runs/11268216954/job/31336270193?pr=98

[drkameleon]

@drkameleon CI now fails: https://github.com/exercism/arturo/actions/runs/11268216954/job/31336270193?pr=98

I'm looking into it!

Btw:

Not sure if I'm totally missing the point, but for the Github workflows, we could simply add the Arturo action like:

- name: Install Arturo
  uses: arturo-lang/arturo-action@main
  with: 
      token: ${{ secrets.GITHUB_TOKEN }}

and the binary should be available in all subsequent steps.

Do you think it would make sense for me to edit the Test workflow? If so, I'll do it right now! 😉

[drkameleon]

Just uncommented the "curl install" line (it has supposedly been fixed as of today) and commented out the call to download_nightly, since we won't need it. (and most likely it is to be removed altogether)

[ErikSchierboom]

Do you think it would make sense for me to edit the Test workflow? If so, I'll do it right now! 😉

Yeah that would be nice.

[ErikSchierboom]

(and most likely it is to be removed altogether)

If CI passes, then the old code can be removed.

[drkameleon]

OK. So, let's see... I think I've fixed the installation script... and it should be working as it should.

If the CI succeeds (just to make sure the installation script works), I would then change the Test workflow so that it uses our existing action (for CI purposes, I think this is even more efficient).

And if it succeeds again, we'll be good to go, by any possible definition 😉

[ErikSchierboom]

Agreed!

[ErikSchierboom]

You've fixed CI, only it turns out that one of our example solutions doesn't work with the latest version :) @BNAndras any idea?

[drkameleon]

The culprit is apparently this:
https://github.com/exercism/arturo/blob/main/exercises/practice/largest-series-product/.meta/src/example.art#L12

And related to how we handle the product of empty blocks:
arturo-lang/arturo#1693

which we have fixed to be in line with most common approaches to the issue:

[drkameleon]

I replaced the Arturo action as we said and tried to fix the error above. Let's see...
Unless I've totally messed up sth trivial, we should be good to go!

[ErikSchierboom]

Perfect! Thanks

[drkameleon]

Perfect! Thanks

Awesome! 😉