Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Add support for wash app validate #2009

Closed
2 of 11 tasks
vados-cosmonic opened this issue Apr 29, 2024 · 1 comment
Closed
2 of 11 tasks

[FEATURE] Add support for wash app validate #2009

vados-cosmonic opened this issue Apr 29, 2024 · 1 comment
Labels
enhancement New feature or request ux Issues related to User eXperience wadm Related to WADM wash-cli Issues pertaining to the wash CLI

Comments

@vados-cosmonic
Copy link
Contributor

Affected project(s)

  • documentation
  • examples
  • wasmCloud host
  • wasmCloud CLI (wash)
  • wasmCloud dashboard UI (washboard)
  • capability providers
  • provider bindgen
  • control interface client
  • other / not sure

Is your feature request related to a problem? Please describe.

When writing/building a WADM manifest, we should make it easier for people to validate whether the manifests have any obvious errors.

Like most other manifest formats, while the information is structurally checked, it is not semantically checked by the tools that are using it -- having some easily callable utility functions that check a manifest to see if it's reasonably configured (possibly emitting warnings) would be useful.

Describe the solution you'd like

To solve this we should:

  • Add and expose some WADM manifest semantic checking functionality to wadm
  • Expose manifest semantic checking functionality in wash and other tools that may benefit from it (might make sense to issue warnings based on validation output)

Note that some structures may belong in core rather than wadm so they can be more easily accessible.

Describe alternatives you've considered

N/A

Additional context

No response

@vados-cosmonic vados-cosmonic added enhancement New feature or request wash-cli Issues pertaining to the wash CLI wadm Related to WADM ux Issues related to User eXperience labels Apr 29, 2024
@vados-cosmonic
Copy link
Contributor Author

This is done! wash app validate was merged and will be released in the next version of wash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux Issues related to User eXperience wadm Related to WADM wash-cli Issues pertaining to the wash CLI
Projects
None yet
Development

No branches or pull requests

1 participant