Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

Update julia.toml #89

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

logankilpatrick
Copy link

Bump Julia Version to 1.4.1

Bump Julia Version to 1.4.1
Copy link
Contributor

@dan-stowell dan-stowell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good & thank you! I'm happy to merge once I know prybar can run against julia 1.4.1: replit/prybar#52

@logankilpatrick
Copy link
Author

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

@dan-stowell
Copy link
Contributor

@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?

@logankilpatrick
Copy link
Author

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).

@dan-stowell
Copy link
Contributor

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.

@logankilpatrick
Copy link
Author

Okay, it should be good:

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

Copy link
Contributor

@dan-stowell dan-stowell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 / I'll merge once the tests are green!

@logankilpatrick
Copy link
Author

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

@amasad
Copy link
Member

amasad commented Aug 24, 2020

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

@logankilpatrick
Copy link
Author

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

@logankilpatrick
Copy link
Author

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
Copy link
Author

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
Copy link
Author

logankilpatrick commented Sep 11, 2020

Bumping this @amasad

@dan-stowell
Copy link
Contributor

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!

@logankilpatrick
Copy link
Author

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

@dan-stowell
Copy link
Contributor

@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. #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.

@logankilpatrick
Copy link
Author

@dan-stowell Hey! Any update on this?

@dan-stowell
Copy link
Contributor

@logankilpatrick I merged dcdaa0f 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.

@logankilpatrick
Copy link
Author

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.

@dan-stowell
Copy link
Contributor

@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): replit/prybar#52 followed by the PR on polygott.

@logankilpatrick
Copy link
Author

logankilpatrick commented Feb 11, 2021 via email

@fingolfin
Copy link

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants