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

Migrate Alacritty config from TOML to YAML format #66

Conversation

RazoBeckett
Copy link
Contributor

@RazoBeckett RazoBeckett commented Sep 13, 2024

Pull Request

Title

Migrate Alacritty config from TOML to YAML format

Type of Change

  • New feature
  • Refactoring
  • Hotfix

Description

Converted Alacritty config back to YAML format for compatibility with systems using older Alacritty versions

Additional Information:

Merge this #367 first on Linutil.

Issue related to PR

resolves the issue: #364 on linutil.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • My changes generate no errors/warnings/merge conflicts.

Converted Alacritty config back to YAML format for compatibility with
systems using older Alacritty versions
@ghost
Copy link

ghost commented Sep 15, 2024

@RazoBeckett Alacritty prefers toml over yaml file format. With a yml version of my toml file, this shows in the terminal.

image

@RazoBeckett
Copy link
Contributor Author

RazoBeckett commented Sep 15, 2024

@mashrukk ya i know it does convert it to TOML using the alacritty migrate command. check the PR #367

@ghost
Copy link

ghost commented Sep 15, 2024

@RazoBeckett Oh right, that makes sense. He has to merge both PRs otherwise this won't work

@RazoBeckett
Copy link
Contributor Author

@RazoBeckett Oh right, that makes sense. He has to merge both PRs otherwise this won't work

YES same thing is mentioned in the Description body of both PRs i hope he reads and then merge or take action on PR.

@ChrisTitusTech
Copy link
Owner

Ahhh let those old men suffer.... TOML FOR LIFE... YAML is for LOSERS!

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

Successfully merging this pull request may close these issues.

2 participants