Add a setup script to install Julia + IJulia

[yuvipanda]

Describe your changes

Based on #1926 (comment),
move Julia installation to a common script.

Issue ticket if applicable

Checklist (especially for first-time contributors)

• I have performed a self-review of my code
• If it is a core feature, I have added thorough tests
• I will try not to use force-push to make the review process easier for reviewers
• I have updated the documentation for significant changes

[yuvipanda]

@mathbunnyru So I added the script, but am not sure how exactly to get it into both the julia and datascience notebooks. I've proposed one way here, stolen from @cboettig's ideas in rocker (https://github.com/rocker-org/rocker-versioned2/blob/965e5a8dbb493dfced6f1463ae7f1f5cd53987e5/dockerfiles/r-ver_4.3.1.Dockerfile#L19). This also allows easier extension downstream - for example, if someone wants to add julia to a spark image, for example. They can just extend and call install-julia.bash. LMK if this sounds good to you?

[mathbunnyru]

@mathbunnyru So I added the script, but am not sure how exactly to get it into both the julia and datascience notebooks. I've proposed one way here, stolen from @cboettig's ideas in rocker (https://github.com/rocker-org/rocker-versioned2/blob/965e5a8dbb493dfced6f1463ae7f1f5cd53987e5/dockerfiles/r-ver_4.3.1.Dockerfile#L19). This also allows easier extension downstream - for example, if someone wants to add julia to a spark image, for example. They can just extend and call install-julia.bash. LMK if this sounds good to you?

I totally agree with this approach and I actually proposed it as well in my comment in a previous PR 😃

[yuvipanda]

@mathbunnyru great! waiting for tests to complete, but what do you think of this? :)

[mathbunnyru]

Looks really nice.
I would like to propose to move the Julia version handling to the bash script as well (and remove it from both Dockerfiles)
Something like - if environment variable is set, use it. If not, use the default version defined in the bash script.

[yuvipanda]

@mathbunnyru done!

[mathbunnyru]

Looks really good now.
Just a few minor things to cleanup and we can merge this :)

[yuvipanda]

hmm, it looks like you need to explicitly approve the GitHub run, @mathbunnyru! Not sure why that happened suddenly.

[mathbunnyru]

The last commit was from the pre-commit.ci[bot] (run pre-commit install in the folder with this repo, your life will be easier 🙂 ) .
And we use self-hosted runners here, they are not isolated, so I only allow actions to be run either by members of the organization or I have to run them explicitly, when there is an outside contributor.

[yuvipanda]

Yeah, been relying on pre-commit.ci because I fucked up my local node install bad enough that prettier won't run anymore and am too lazy to fix it :D

Oooh, TIL this uses self hosted runners! Thank you :)

[yuvipanda]

Hooray, tests pass!

[yuvipanda]

Yay, thank you @mathbunnyru!