Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

simplify validation errors#104

Open
pomdtr wants to merge 2 commits intomainfrom
improve-validation-errors
Open

simplify validation errors#104
pomdtr wants to merge 2 commits intomainfrom
improve-validation-errors

Conversation

@pomdtr
Copy link
Contributor

@pomdtr pomdtr commented May 3, 2023

Our current validation error message are hard to craft, and still do not provide a good user experience.

Error: Spacefile
  L Micro 'static-app'
    L Micro 'static-app'
      L Micro 'static-app'
        L missing properties: 'serve'

Ideally, we want something like like this

image

However, it is not trivial to achieve. In the meantime, I think this is a good middleground:

Error: validation failed:
L /micros/static-app: missing properties: 'serve'

@pomdtr pomdtr requested a review from BetaHuhn May 5, 2023 06:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants