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.

Sign up for GitHub

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

Jump to bottom

Use yaml package for formatting #983

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Use yaml package for formatting #983

wants to merge 2 commits into from

Commits on Jul 24, 2024

  1. Use yaml package for formatting 

    This replaces Prettier as the LSP formatting solution with the `yaml`
package. The `yaml` package can format YAML just fine, and we already
have it as a dependency. Prettier is a big dependency.

If people want to use Prettier, other Prettier integrations are probably
better suited for them. For example, the YAML language server doesn’t
respect Prettier configuration files.

This is a proof of concept. I tried to map existing options to the new
implementation. A more ideal solution might be to change the formatting
options.

Refs redhat-developer#933
    @remcohaszing
    remcohaszing committed Jul 24, 2024
    Configuration menu
    Copy the full SHA
    db44f9b View commit details
    Browse the repository at this point in the history

  2. Update to yaml 2.5.0

    @remcohaszing
    remcohaszing committed Jul 24, 2024
    Configuration menu
    Copy the full SHA
    8b523e5 View commit details
    Browse the repository at this point in the history