Skip to content

chore: Config Updates ( TOML / JSON / YAML )#205

Merged
lxfontes merged 6 commits intomainfrom
lxfontes/yaml-config
Dec 18, 2025
Merged

chore: Config Updates ( TOML / JSON / YAML )#205
lxfontes merged 6 commits intomainfrom
lxfontes/yaml-config

Conversation

@lxfontes
Copy link
Member

@lxfontes lxfontes commented Dec 11, 2025

More config formats

This PR aligns wash config with its underlying crate: figment

With these changes, wash config:

  • Can be expressed as JSON / YAML / TOML
  • Match file extensions ( .json, .yaml | .yml, .toml )

CLI Context ( Execution Directory )

Instead of requiring each wash command to manage project path locations, we now centralize all project path operations in global arguments and follow similar gnutools expectations.

We take the current directory into account and try to find the .wash directory walking back the directory tree.
If we can't find the project root, we use the current directory.

We also expose a -C flag where users can force a project path. This is useful in CI environments & situations where users cant change directories prior calling wash.

Auto Update

Auto update file management was ☠️ . It would never expire the local cache, meaning you'd never be able to upgrade or refresh the cache ( as we don't provide a cache clear option ).

This PR makes is so the local cache for release-list is refreshed at most once in 24 hours.

Test fixes

We were changing the current directory in many parallel tests. This could cause tests to override each others files & fail.
integration_wit.rs now executes tests in sequence.

Signed-off-by: Lucas Fontes <lucas@cosmonic.com>
Signed-off-by: Lucas Fontes <lucas@cosmonic.com>
Signed-off-by: Lucas Fontes <lucas@cosmonic.com>
Signed-off-by: Lucas Fontes <lucas@cosmonic.com>
Signed-off-by: Lucas Fontes <lucas@cosmonic.com>
@lxfontes lxfontes changed the title chore: Moving config from JSON to YAML chore: Config Updates ( TOML / JSON / YAML ) Dec 18, 2025
Signed-off-by: Lucas Fontes <lucas@cosmonic.com>
@lxfontes lxfontes marked this pull request as ready for review December 18, 2025 21:15
@lxfontes lxfontes requested a review from a team as a code owner December 18, 2025 21:15
@lxfontes lxfontes added this pull request to the merge queue Dec 18, 2025
Merged via the queue into main with commit ea227bb Dec 18, 2025
9 checks passed
@lxfontes lxfontes deleted the lxfontes/yaml-config branch December 18, 2025 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants