Skip to content
New issue

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

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

Already on GitHub? # to your account

Update README.md #475

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shantanu-sardesai
Copy link

Hi, I started learning OCaml today and was installing utop when I noticed a minor inconsistency in between the directions given in the utoprc-dark file and the README on the location to store the theme information.

The ones in the utoprc-dark worked for me, so creating a PR to update the path in the README.

Fix the configuration file path for utop theme.
@emillon
Copy link
Collaborator

emillon commented Jan 15, 2024

No, utop should load the files from the path in README since #431. What version of utop are you using?

@shantanu-sardesai
Copy link
Author

shantanu-sardesai commented Jan 15, 2024

Should be the latest one since I Installed it this morning using the command from the README.

A screenshot of my utop version (v2.13.1):

image

could it be that I misunderstood what the effects of adding the code from utoprc-dark are?

It made some minor changes in the color being displayed in my repl after I pasted the contents in ~/.config/utoprc, didn't do the same when I pasted it in the ~/.config/utop/utoprc first.

Without any configuration:

image

With configuration in ~/.config/utoprc:

image

With configuration in ~/.config/utop/utoprc (Same as without configuration):

image

My utoprc file:
image

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants