Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

Add a waypoint fmt command #1037

Merged
merged 7 commits into from
Feb 5, 2021
Merged

Add a waypoint fmt command #1037

merged 7 commits into from
Feb 5, 2021

Conversation

mitchellh
Copy link
Contributor

Fixes #705

This adds a waypoint fmt command to format waypoint.hcl files. This is a simplified version of terraform fmt: https://www.terraform.io/docs/cli/commands/fmt.html and the underlying logic is identical.

@mitchellh mitchellh added this to the 0.3.0 milestone Jan 28, 2021
@mitchellh mitchellh requested a review from a team January 28, 2021 22:46
@github-actions github-actions bot added the core label Jan 28, 2021
@mitchellh
go.mod
26636b7 
go.*
@mitchellh
internal/config: Format function
05f91a2
@mitchellh
internal/cli: initial fmt
7d38ef5 
internal/cli: add fmt

internal/cli: fmt should print not println
briancain
briancain suggested changes Jan 28, 2021
View reviewed changes
Copy link
Member

@briancain briancain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well for me! Just a quick note about the -write behavior to match terraform fmt.

internal/cli/fmt.go Show resolved Hide resolved
@mitchellh mitchellh merged commit 951e553 into main Feb 5, 2021
@mitchellh mitchellh deleted the f-fmt branch February 5, 2021 21:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@briancain briancain briancain approved these changes

Assignees
No one assigned
Labels
core
Projects
None yet
Milestone
0.3.0
Development

Successfully merging this pull request may close these issues.

Have a waypoint fmt to format the waypoint.hcl
2 participants
@mitchellh @briancain