FuelLabs / FuelLabs/sway

ci(fmt): add formatting regression tests to the `e2e` test harness

Open
#4,249 1 comment 0 reactions 0 assignees View on GitHub
ci formatter
Dominant language
Rust
Stars
61.4k
Forks
5.4k
Avg merge
3h 33m
Merged PRs (30d)
4

Description

With formatting, the more checks the better - adding all the projects within should_pass to fmt checks is a decent way to stress test the formatter and keep it from regressing.

> My thinking is that, we know they should already at least always be parsing successfully, so we should be able to reliably check their formatting too.

_Originally posted by @mitchmindtree in https://github.com/FuelLabs/sway/issues/4192#issuecomment-1461392582_

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the `e2e` test harness and its `should_pass` project set first. Extend the formatting checks to cover every project in `should_pass`, then verify the harness catches formatting regressions while preserving successful parsing.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
ci-cd, compilers, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.