EnergySystemsModellingLab / EnergySystemsModellingLab/MUSE2

Replace `prettier` pre-commit hook

Open
#582 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
8
Forks
5
Avg merge
1d 2m
Merged PRs (30d)
37

Description

The `prettier` pre-commit has seemingly been dead for a while now: https://github.com/EnergySystemsModellingLab/MUSE_2.0/issues/581

While it works for now, we probably shouldn't rely on this. Let's move to using other hooks to format JSON and YAML. See example in `python-template` repo: https://github.com/ImperialCollegeLondon/python-template/blob/main/.pre-commit-config.yaml

When I last checked, I couldn't get pre-commit to work in a container using our `Dockerfile`, I think because of some nodejs-related issue. Moving to Python-based hooks might also fix this problem.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.