sillsdev / sillsdev/languageforge-lexbox

Run Prettier over the whole viewer, not just components

Open Beginner friendly
#2,474 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C#
Stars
9
Forks
8
Avg merge
2d 13h
Merged PRs (30d)
49

Description

Prettier in frontend/viewer is currently scoped to src/lib/components/** only. frontend/viewer/.prettierignore has a "Temporarily ignore everything except components" block that excludes the rest of the viewer, so format:check (enabled in CI by the PR this issue follows) only covers the components directory.

Follow-up: widen Prettier to the whole viewer, using the same pattern as the components rollout (#2151):

  1. Remove the temporary ignore block from .prettierignore.
  2. Run pnpm run format as a single dedicated commit.
  3. Add that commit to .git-blame-ignore-revs.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with frontend/viewer/.prettierignore and remove the temporary block that excludes everything except components. Run pnpm run format and keep the formatting changes in one dedicated commit, then add that commit to .git-blame-ignore-revs. Done means format:check covers the whole viewer and the ignore-revs entry is present.

Written by the indexing model from the issue text.

Assessment

Domain
frontend, tooling
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
78/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.