polygott icon indicating copy to clipboard operation
polygott copied to clipboard

Update julia.toml

Open logankilpatrick opened this issue 5 years ago • 20 comments

Bump Julia Version to 1.4.1

logankilpatrick avatar May 08 '20 14:05 logankilpatrick

@dan-stowell @amasad Can we get a merge on this? Julia 1.5 is out but I'm bumping this incrementally.

logankilpatrick avatar Aug 14 '20 13:08 logankilpatrick

@logankilpatrick can you you set the PRYBAR_TAG in the Dockerfile to circleci_pipeline_89_build_96, run make test-julia, then push that commit and I'll merge?

dan-stowell avatar Aug 14 '20 14:08 dan-stowell

What dir do I have to be in to run that command? Running from the root does not work (I am a big Docker noob).

logankilpatrick avatar Aug 14 '20 14:08 logankilpatrick

What dir do I have to be in to run that command? Running from the root does not work (I am a big Docker noob).

From the polygott dir, you can run make test-julia and it should build a julia image & run the tests.

dan-stowell avatar Aug 14 '20 15:08 dan-stowell

Okay, it should be good:

Successfully built 8ff8b7cdc88c
Successfully tagged polygott-julia:latest
docker run polygott-julia bash -c polygott-self-test
✓ julia:hello

logankilpatrick avatar Aug 14 '20 15:08 logankilpatrick

Wonderful, once this goes through, I will open the PR to bump to the latest Julia version (v1.5).

logankilpatrick avatar Aug 14 '20 15:08 logankilpatrick

@logankilpatrick just checking in here -- is the test failure related to your change?

amasad avatar Aug 24 '20 03:08 amasad

I tested locally per what I commented above so I'm not sure.

logankilpatrick avatar Aug 24 '20 04:08 logankilpatrick

It worked again when I tested locally:

Successfully built 2922b9ce9d41
Successfully tagged polygott-julia:latest
docker run polygott-julia bash -c polygott-self-test
✓ julia:hello

logankilpatrick avatar Aug 24 '20 04:08 logankilpatrick

The tests are also failing due to permission issues (according to the build) due to me not being a collaborator on this repo maybe?

logankilpatrick avatar Aug 24 '20 04:08 logankilpatrick

Bumping this @amasad

logankilpatrick avatar Sep 11 '20 13:09 logankilpatrick

Good afternoon @logankilpatrick ! You've probably noticed by now that we've been slow to merge community PRs on polygott. We've decided to focus on giving Repl.it a stable foundation instead of adding features and updating languages. Once we and our community feel comfortable with Repl.it's overall stability, we'll give this change another look. This change, in particular, should be straightforward to get over the finish line. Thank you for contributing!

dan-stowell avatar Oct 01 '20 19:10 dan-stowell

Hey all, any update on this? Would be great if Repl.it supported the most recent Julia version.

logankilpatrick avatar Nov 20 '20 03:11 logankilpatrick

@logankilpatrick, apologies for the long delay on a straightforward pull request. We've been focused on overall Replit stability the past few months. We're now in a position to roll out clearly beneficial Polygott changes. https://github.com/replit/polygott/pull/185 upgrades to 1.4.1 (the build should be green on that branch), and then we can look at getting to 1.5.3 right after that.

dan-stowell avatar Dec 09 '20 03:12 dan-stowell

@dan-stowell Hey! Any update on this?

logankilpatrick avatar Dec 23 '20 14:12 logankilpatrick

@logankilpatrick I merged https://github.com/replit/polygott/commit/dcdaa0ff51114d3725fc036540e9eb6506afd15f over the weekend. It'll go out with the next polygott release this week. I'll leave this PR open to update you on when it's rolling out.

dan-stowell avatar Feb 08 '21 22:02 dan-stowell

Great! @dan-stowell if we merge in Julia 1.5.3, can we get that in for the release? I can submit a Pr tonight.

logankilpatrick avatar Feb 08 '21 22:02 logankilpatrick

@logankilpatrick Julia 1.4.1 is out if you create a new julia_beta repl: https://repl.it/l/julia_beta

(The console header says 1.3.1, but if you run julia --version in the shell you'll see it's actually 1.4.1. I'm pushing a config change to correct the console header shortly.)

And, yes! If you open a pull request upgrading to Julia 1.5.3, we can release that next. It'd be most helpful to get a pull request up against prybar (much like I did for the Julia 1.4.1 upgrade): https://github.com/replit/prybar/pull/52 followed by the PR on polygott.

dan-stowell avatar Feb 10 '21 14:02 dan-stowell

Great, just opened PR’s in both places. Thanks again!

On Feb 10, 2021, at 6:27 AM, Dan Stowell [email protected] wrote:

@logankilpatrick https://github.com/logankilpatrick Julia 1.4.1 is out if you create a new julia_beta repl: https://repl.it/l/julia_beta https://repl.it/l/julia_beta (The console header says 1.3.1, but if you run julia --version in the shell you'll see it's actually 1.4.1. I'm pushing a config change to correct the console header shortly.)

And, yes! If you open a pull request upgrading to Julia 1.5.3, we can release that next. It'd be most helpful to get a pull request up against prybar (much like I did for the Julia 1.4.1 upgrade): replit/prybar#52 https://github.com/replit/prybar/pull/52 followed by the PR on polygott.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/replit/polygott/pull/89#issuecomment-776743753, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIPN5XQIMGCYQF4XFXXWBXLS6KJTVANCNFSM4M4GS4PQ.

logankilpatrick avatar Feb 11 '21 14:02 logankilpatrick

I think PR #220 and https://github.com/replit/prybar/pull/52 mean this PR is obsolete. And there is also PR #222 by now

fingolfin avatar Oct 27 '21 17:10 fingolfin