unisonweb/unison

Pretty Printer inconsistent with tuple spacing in list items

Open

#5,004 opened on May 23, 2024

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Haskell (6,624 stars) (304 forks)batch import
help wantedpretty-printer

Description

image

Notice that the tuples that have a line break between items have a space between ( and the first element, while the tuples without line breaks have no space between ( and the first element.

Contributor guide

Tech stack
None
Domain
toolingdeveloper experience
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Haskell knowledgeFamiliarity with Unison syntax
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
Examine the pretty printer logic for tuples in list contexts. Look for where spacing is added after '(' depending on line breaks. The fix likely involves ensuring consistent spacing regardless of line breaks. Check the relevant source files in the 'unison' repository under the pretty printer directory.