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

Configuration migration creates an invalid config file #5909

Closed
1 task done
Salamandar opened this issue Nov 19, 2024 · 1 comment
Closed
1 task done

Configuration migration creates an invalid config file #5909

Salamandar opened this issue Nov 19, 2024 · 1 comment
Assignees
Labels
🐛 bug Something isn't working

Comments

@Salamandar
Copy link

Salamandar commented Nov 19, 2024

Code of Conduct

  • I agree to follow this project's Code of Conduct

What happened?

When installing OMP v24.6.4, it tried to convert my configuration file from v2 to v3. And i found myself with a "CONFIG ERROR" on my prompt.

After some tinkering, I found out that it removed double quotes on most strings in the yaml file, and replaced

transient_prompt:
  filler: "-"

with

transient_prompt:
  filler: -

and this is not a string, i'm thinking this is understood as a list.

Adding back the double quotes solves the issue.

What OS are you seeing the problem on?

Linux

Which shell are you using?

zsh

@Salamandar Salamandar added the 🐛 bug Something isn't working label Nov 19, 2024
@JanDeDobbeleer
Copy link
Owner

JanDeDobbeleer commented Nov 19, 2024

@Salamandar ah, that's a YAML thing I don't think I can solve as some characters have special meanings. There's others as well that have the same effect, I'll check but I think it's a YAML spec issue.

JanDeDobbeleer added a commit that referenced this issue Jan 3, 2025
JanDeDobbeleer added a commit that referenced this issue Jan 3, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants