Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DX: install Julia through juliaup #201

Merged
merged 7 commits into from
Aug 4, 2023
Merged

DX: install Julia through juliaup #201

merged 7 commits into from
Aug 4, 2023

Conversation

redeboer
Copy link
Member

Thanks to a tip from @Moelf, Julia is now installed within the Conda environment through juliaup. Installation instructions have been updated accordingly.

@redeboer redeboer added the 🖱️ DX Improvements to the Developer Experience label Jul 28, 2023
@redeboer redeboer self-assigned this Jul 28, 2023
@ComPWA ComPWA deleted a comment from review-notebook-app bot Jul 28, 2023
@Moelf
Copy link
Contributor

Moelf commented Jul 28, 2023

The instructions guide users to install 1.7 but the notebook and Manifest has 1.9

Maybe just guide user to install 1.9?

@redeboer
Copy link
Member Author

redeboer commented Jul 28, 2023

The instructions guide users to install 1.7 but the notebook and Manifest has 1.9

Ah thanks, well spotted. For the home-folder/system-wide install, right? Fixed now.

But it does point to the fact that these instructions should perhaps just point to official Julia install guides... Anyway, in first instance, the reader just sees the juliaup instructions (temporary preview here):
image

docs/develop.md Outdated Show resolved Hide resolved
@redeboer redeboer enabled auto-merge (squash) August 4, 2023 19:28
@redeboer redeboer merged commit a63d855 into main Aug 4, 2023
9 checks passed
@redeboer redeboer deleted the juliaup branch August 4, 2023 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖱️ DX Improvements to the Developer Experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants